This is the mail archive of the crossgcc@sourceware.org mailing list for the crossgcc project.

See the CrossGCC FAQ for lots more information.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch for building Java compiler for version 4.3.0+


Andy,
All,

On Tuesday 06 January 2009 16:53:54 Andy Johnson wrote:
> > OK. Then we'll need a silent option:
> --SNIP--
> I have not figured which file to put the configure option in
> and have it work.  The two "depends" are set in different files.
> No matter where I put it CC_LANG_JAVA_USE_ECJ is never set in
> the .config file.  The build process dies when it tries to test
> that variable.

The kernel mconf prefixes all config option with CONFIG_, the one in
crosstool-NG uses CT_ as prefix, thus an option named CC_LANG_JAVA_USE_ECJ
will be available as CT_CC_LANG_JAVA_USE_ECJ

Something like the attached patch?

> I don't know how to do a list of possible ftp sites to get the
> ecj-latest.jar file, but another is at ftp://sourceware.org/pub/java/.

Added! :-)

> Also, there is an additional configure option for gcc that I
> added through menuconfig (--enable-java-maintainer-mode) that
> I'm not sure how you would like to handle.  I'm guessing that
> you would rather put it in a config file rather than code it
> into gcc.sh.

What does it do? Is it sensible to have it:
1- always?
2- upon request (config option)?
3- only occasionally (set it in CC_EXTRA_CONFIG)?

As I understand the option by its name, 3 seems the best way.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +0/33 662376056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| --==< ^_^ >==-- `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
`------------------------------^-------^------------------^--------------------'
Index: ct-ng.trunk.ecj/scripts/build/cc/gcc.sh
===================================================================
--- ct-ng.trunk.ecj/scripts/build/cc/gcc.sh	(revision 1294)
+++ ct-ng.trunk.ecj/scripts/build/cc/gcc.sh	(working copy)
@@ -12,12 +12,21 @@
     # subdirectory! You bastard!
     CT_GetFile "gcc-${CT_CC_VERSION}"   \
                {ftp,http}://ftp.gnu.org/gnu/gcc{,{,/releases}/gcc-${CT_CC_VERSION}}
+
+    if [ "${CT_CC_LANG_JAVA_USE_ECJ}" = "y" ]; then
+        CT_GetFile ecj-latest .jar ftp://gcc.gnu.org/pub/java   \
+                                   ftp://sourceware.org/pub/java
+    fi
 }
 
 # Extract gcc
 do_cc_extract() {
     CT_Extract "gcc-${CT_CC_VERSION}"
     CT_Patch "gcc-${CT_CC_VERSION}"
+
+    if [ "${CT_CC_LANG_JAVA_USE_ECJ}" = "y" ]; then
+        CT_DoExecLog ALL cp -v "${CT_TARBALLS_DIR}/ecj-latest.jar" "${CT_SRC_DIR}/gcc-${CT_CC_VERSION}/ecj.jar"
+    fi
 }
 
 #------------------------------------------------------------------------------
Index: ct-ng.trunk.ecj/config/cc.in
===================================================================
--- ct-ng.trunk.ecj/config/cc.in	(revision 1294)
+++ ct-ng.trunk.ecj/config/cc.in	(working copy)
@@ -65,6 +65,12 @@
       Only select this if you know that your specific version of the
       compiler supports this language.
 
+config CC_LANG_JAVA_USE_ECJ
+    bool
+    default y
+    depends on CC_LANG_JAVA
+    depends on CC_GCC_4_3_or_later
+
 config CC_LANG_ADA
     bool
     prompt "ADA (EXPERIMENTAL)"

--
For unsubscribe information see http://sourceware.org/lists.html#faq

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]