This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


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: retracting my retraction [patch included]


Hi All,

>Sorry about the delay on reading this.
>I finally read this whole thread.
>
>Adam>   -         GCJ="${target_noncanonical}-gcj -B`${PWDCMD-pwd}`/"
>Adam>   +         GCJ="$built_gcc_dir/gcj -B$built_gcc_dir/"
>
>I think this patch is probably causing problems for Marco.
>However I'm not certain.

Whoa. Did this actually get committed? In one of the follow-ups,
I pointed out that this was wrong and Andrew came around and
agreed with me. This hadn't gotten committed to the ecj branch
at the time that I was working on an alternative patch.

OK. Looking at SVN, I see this has been committed. This is wrong
for the reasons I mentioned in the followup and should be reverted:

http://gcc.gnu.org/ml/java-patches/2006-q4/msg00313.html

For some reason, this was applied against the mainline and not to
gcj-eclipse-merge-branch. I would have never thought to check
for this scenario when I saw Marco's emails.

-- Mohan
http://www.thisiscool.com/
http://www.animalsong.org/




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