build problem: sed -E is unknown switch

[GoogleCodeExporter]

In sed version 4.1.5 there is no "-E" switch. I assume this is a typo and
it should read "-e".

Original issue reported on code.google.com by everf...@gmail.com on 13 Jun 2009 at 8:31

[GoogleCodeExporter]

Aaargh !
That'd be a FreeBSD-only switch !
"The -E -, -a and -i options are non-standard" (see 
http://www.gsp.com/cgi-bin/man.cgi?
section=1&topic=sed)

Is there a way to know in a script what the operating system is ?

Thanks for the report !

Original comment by olivier.chafik@gmail.com on 13 Jun 2009 at 9:14

• Changed state: Accepted
• Added labels: Priority-Critical
• Removed labels: Priority-Medium

[GoogleCodeExporter]

Mhm that's really bad.
Perhaps switching to a different build system could help in this case. (maven, 
ant, ...)
Especially maven is popular and would make building for external people easier 
while
additionally avoiding the bundling of dependencies in the scm.

Original comment by everf...@gmail.com on 13 Jun 2009 at 10:03

[GoogleCodeExporter]

I removed the -E switch and it *appears* to work...
I'd be open to switching to Maven if someone contributed the scripts ;-)
I don't know how difficult it is to create non-standard builds (I need to 
compile the runtime in -target 1.5, the 
rest as 1.6, I merge JNA and overwrite one of its classes - Structure.java, + 
I'm not using their trunk right now...)
Cheers

Original comment by olivier.chafik@gmail.com on 22 Jun 2009 at 9:50

• Changed state: Fixed

[GoogleCodeExporter]

Indeed, looks good. An invalid import slipped in, though (at least I guess so)
Fix:

Index: sources/com/ochafik/lang/jnaerator/DeclarationsConverter.java
===================================================================
--- sources/com/ochafik/lang/jnaerator/DeclarationsConverter.java   (revision 478)
+++ sources/com/ochafik/lang/jnaerator/DeclarationsConverter.java   (working copy)
@@ -46,7 +46,6 @@
 import com.ochafik.util.string.StringUtils;
 import com.sun.jna.*;
 import com.sun.jna.Pointer;
-import com.sun.tools.corba.se.idl.InvalidArgument;

 import static com.ochafik.lang.jnaerator.parser.ElementsHelper.*;
 public class DeclarationsConverter {

Original comment by everf...@gmail.com on 23 Jun 2009 at 9:36