Working on Spatial index

View: New views
3 Messages — Rating Filter:   Alert me  

Working on Spatial index

by leszek-6 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hello everybody,

As this is the first time I use this mailing list, let me introduce
myself.
My name is Hanusz Leszek, student in the last year of Computer
Engineering at the ULB university.
I am working with the Spatial Index on eXist for a project for the
university.

I will probably send a few patches in the following weeks.

In attachment is already a very small patch (one character) which
address a little typo.
As this is my first patch, please tell me if there is any rule that
I should follow.

Regards,

Hanusz Leszek


[spatial_hsql_sh_path_typo.patch]

Index: trunk/eXist/extensions/indexes/spatial/hsql.sh
===================================================================
--- trunk/eXist/extensions/indexes/spatial/hsql.sh (révision 8012)
+++ trunk/eXist/extensions/indexes/spatial/hsql.sh (copie de travail)
@@ -14,7 +14,7 @@
 HSQL_LIB="${EXIST_HOME}/extensions/indexes/spatial/lib"
 
 if [ "x$1" = "x" ]; then
-    HSQL_DATA="${EXIST_HOME}/webapp\WEB-INF/data/spatial_index"
+    HSQL_DATA="${EXIST_HOME}/webapp/WEB-INF/data/spatial_index"
 else
     HSQL_DATA="${EXIST_HOME}/$1"
 fi


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Exist-open mailing list
Exist-open@...
https://lists.sourceforge.net/lists/listinfo/exist-open

Re: Working on Spatial index

by bkc :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi,

I am not one of the developers, but I'd like to suggest that you create
a new bug and attach this patch to the bug.

That way it won't get "lost in the list"

The tracker is here
http://sourceforge.net/tracker/?func=browse&group_id=17691&atid=117691



--
Brad Clements,                bkc@...    (315)268-1000
http://www.murkworks.com                         
AOL-IM: BKClements


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Exist-open mailing list
Exist-open@...
https://lists.sourceforge.net/lists/listinfo/exist-open

Re: Working on Spatial index

by Leif-Jöran Olsson :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

- ----- Ursprungligt meddelande -----
@A@ skrev @D@:
| Hello everybody,
|
| As this is the first time I use this mailing list, let me introduce
| myself.
| My name is Hanusz Leszek, student in the last year of Computer
| Engineering at the ULB university.
| I am working with the Spatial Index on eXist for a project for the
| university.
|
| I will probably send a few patches in the following weeks.

OK. Good to hear. Welcome!

| In attachment is already a very small patch (one character) which
| address a little typo.

Thankyou, it is applied in trunk.

| As this is my first patch, please tell me if there is any rule that
| I should follow.

Well, just small proper procedures really.

1) Check behaviour with trunk head revision and apply the patch against
it if is not a really critical thing.
2) If it is a larger patch, affecting more than one char, attach a
testcase (junit) or at least a minimal reproducible procedure to spot
the different behaviours. This will make the path to applied much
shorter for the patch.

3) Then like Brad added, attach it to the tracker, but choose *the patch
one* like you (or dannes) did in this case of a patch.
Lastly write a mail to this list with the url in it for convenience of
the community (and the devs).

Thanks again,

Leif-Jöran
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iD8DBQFIgdofhcIn5aVXOPIRAj0XAKCQwMmEzQHBhaZXLH27mCsIGcfUmwCgt21O
z4zGz1DegJ0LxqGDWEPHnto=
=lAsX
-----END PGP SIGNATURE-----

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Exist-open mailing list
Exist-open@...
https://lists.sourceforge.net/lists/listinfo/exist-open
LightInTheBox - Buy quality products at wholesale price!