Mao,
thanks for reporting this. on my OSX,
uname -m 2>/dev/null | tr [A-Z] [a-z] | tr -d ' ' and
uname -p 2>/dev/null | tr [A-Z] [a-z] | tr -d ' '
are both producing i386.
I guess since the main case seems to work, switching is safe. We will
test tomorrow, but I am committing the update to accommodate your
feedback, switching the lines,
Index: src/main/distribution/shell-scripts/bin/neo4j
===================================================================
--- src/main/distribution/shell-scripts/bin/neo4j (revision 7096)
+++ src/main/distribution/shell-scripts/bin/neo4j (working copy)
@@ -230,14 +230,14 @@
APP_PLIST=${APP_PLIST_BASE}.plist
else
DIST_ARCH=
- DIST_ARCH=`uname -p 2>/dev/null | tr [A-Z] [a-z] | tr -d ' '`
+ DIST_ARCH=`uname -m 2>/dev/null | tr [A-Z] [a-z] | tr -d ' '`
if [ "X$DIST_ARCH" = "X" ]
then
DIST_ARCH="unknown"
fi
if [ "$DIST_ARCH" = "unknown" ]
then
- DIST_ARCH=`uname -m 2>/dev/null | tr [A-Z] [a-z] | tr -d ' '`
+ DIST_ARCH=`uname -p 2>/dev/null | tr [A-Z] [a-z] | tr -d ' '`
fi
case "$DIST_ARCH" in
'athlon' | 'i386' | 'i486' | 'i586' | 'i686')
Committed as of rev. 7412, thanks for sharing!
Cheers,
/peter neubauer
GTalk: neubauer.peter
Skype peter.neubauer
Phone +46 704 106975
LinkedIn http://www.linkedin.com/in/neubauer
Twitter http://twitter.com/peterneubauer
http://www.neo4j.org - Your high performance graph database.
http://www.thoughtmade.com - Scandinavia's coolest Bring-a-Thing party.
On Wed, Dec 1, 2010 at 7:41 PM, Mao PU <[email protected]> wrote:
> I am just testing the current Mileston V1.2.M04 for unix. I have an issue
> with the definition of $DIST_ARCH in 233 and 240:
>
> Regarding the http://en.wikipedia.org/wiki/Uname and as much as I understood
> from the code of uname, the `uname -m` switch might result in less confusing
> messages than -p can. On my machine building coreutils-8.7 from
> http://ftp.gnu.org/gnu/coreutils/, `uname -p` produces the output "Intel(R)
> Core(TM)2 CPU T5600 @ 1.83GHz".
>
> I think there are two options:
> 1: adding some logic that will translate "Intel Core" to something like
> "i686"
> 2: switch statements in bin/neo4j:233 and bin/neo4j:240 since `-m` seems
> to have a higher chance to return the right thing.
>
> Regards
> Mao PU
> _______________________________________________
> Neo4j mailing list
> [email protected]
> https://lists.neo4j.org/mailman/listinfo/user
>
_______________________________________________
Neo4j mailing list
[email protected]
https://lists.neo4j.org/mailman/listinfo/user
