public inbox for java@gcc.gnu.org
 help / color / mirror / Atom feed
From: Matthias Klose <doko@ubuntu.com>
To: java@gcc.gnu.org
Subject: Re: unable to regenerate the pregenerated libjava header files on   the  trunk
Date: Sat, 13 Mar 2010 14:24:00 -0000	[thread overview]
Message-ID: <4B9BA018.5000205@ubuntu.com> (raw)
In-Reply-To: <4B8A4453.4030606@redhat.com>

On 28.02.2010 11:24, Andrew Haley wrote:
> On 02/27/2010 04:55 PM, Matthias Klose wrote:
>> this is http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42676, now
>> downgraded to P4.  At least for Debian, this would mean not shipping gcj
>> anymore in the main section of the archive (cannot be built from
>> source). Is this a restriction for the upstream release as well? Can
>> this be reproduced by others?
>
> I think you're bootstrapping with an obsolete version of gjavah.
>
> To see if I'm right, generate the header file for
> javax/xml/xpath/XPathFunctionResolver.  Like this:
>
>   $ gjavah -cni -force -classpath<path>/x86_64-unknown-linux-gnu/libjava/libgcj-4.5.0.jar javax/xml/xpath/XPathFunctionResolver
>
> With the new version of gjavah, you'll get
>
>       namespace xml
>       {
>         namespace namespace$
>
> With the old version you'll get
>
>       namespace xml
>       {
>         namespace namespace
>
> Andrew.

with a current build, I didn't see these changes anymore. A few differences in 
the header files for inner classes are still present, of the form

--- javax/net/ssl/TrustManagerFactory$1.h       (revision 157407)
+++ javax/net/ssl/TrustManagerFactory$1.h       (working copy)
@@ -27,7 +27,7 @@
  public: // actually package-private
    TrustManagerFactory$1();
  public:
-  ::java::lang::Object * run();
+  virtual ::java::lang::Object * run();
    static ::java::lang::Class class$;
  };

These are seen with the updated ecj-latest.jar (based on eclipse-3.5.1).

2010-03-13  Matthias Klose  <doko@ubuntu.com>

         PR libgcj/42676
         * Regenerate .h files for anonymous inner classes.

Checking in, approved on irc by Andrew. Complete diff in the bug report.

   Matthias

      reply	other threads:[~2010-03-13 14:24 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-27 16:56 Matthias Klose
2010-02-28 10:24 ` Andrew Haley
2010-03-13 14:24   ` Matthias Klose [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4B9BA018.5000205@ubuntu.com \
    --to=doko@ubuntu.com \
    --cc=java@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).