[PATCH] Delete GCJ

classic Classic list List threaded Threaded
63 messages Options
1234
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Matthias Klose-6
On 10.10.2016 09:58, Iain Sandoe wrote:

>
>> On 10 Oct 2016, at 05:03, Matthias Klose <[hidden email]> wrote:
>>
>> On 07.10.2016 10:30, Iain Sandoe wrote:
>>>
>>>> On 7 Oct 2016, at 00:58, Matthias Klose <[hidden email]> wrote:
>>>>
>>>> On 06.10.2016 20:00, Mike Stump wrote:
>>>>> On Oct 6, 2016, at 9:56 AM, Rainer Orth <[hidden email]> wrote:
>>>>>> I wouldn't hard-fail, but completely disable objc-gc with an appropriate
>>>>>> warning.  The Objective-C maintainers may have other preferences, though.
>>>>
>>>> I think I can't do that in the top level make file very well (currently I only
>>>> have the pkg-config check there for an early failure, but that check doesn't
>>>> tell me if the library is present for all multilib variants). And I can't check
>>>> for multilibs because I don't know if the bootstrap compiler is multilib aware.
>>>
>>> hrm, so perhaps we need a —with-target-boehm-gc= type arrangement, and it’s the configurer’s responsibility to provide a path with appropriate headers/libs for the multi-lib configuration being attempted.
>>
>> I don't understand what you are proposing here.
>
> given that:
>  auto-detection of the capabilities could be quite difficult (or, in the general case, unachievable) for the reasons you gave.
>  the chosen target libraries might be in a non-standard place.
>
> making it an explicit requirement to point to them, and to ensure that the versions/multi-libs provided are suitable, (by putting —with-target-boehm-gc=/path/to/suitable/) would mean that the dependent configury (for objc-gc) could be just conditional upon the  presence of the "with-target-boehm-gc”.
>
> I suppose that one could make "—with-target-boehm-gc” (no attached path) declare that the library (and requisite mult-lib versions) will be found in the sysroot without any additional work.
>
> The point here was to simplify the dependent configury so that it only needs to test something that the configuring user specifies (i.e. if they specify objc-gc, then they need also to specify the place that the gc lib can be found).
So here is the next proposal, I hope the added documentation in install.texi
makes the usage clear.

<toplevel>

2016-11-19  Matthias Klose  <[hidden email]>

        * Makefile.def: Remove reference to boehm-gc target module.
  * configure.ac: Include pkg.m4, check for --with-target-bdw-gc
        options and for the bdw-gc pkg-config module.
        * configure: Regenerate.
        * Makefile.in: Regenerate.

gcc/

2016-11-19  Matthias Klose  <[hidden email]>

        * doc/install.texi: Document configure options --enable-objc-gc
        and --with-target-bdw-gc.

config/

2016-11-19  Matthias Klose  <[hidden email]>

        * pkg.m4: New file.

libobjc/

2016-11-19  Matthias Klose  <[hidden email]>

        * configure.ac (--enable-objc-gc): Allow to configure with a
        system provided boehm-gc.
        * configure: Regenerate.
        * Makefile.in (OBJC_BOEHM_GC_LIBS): Get value from configure.
        * gc.c: Include system bdw-gc headers.
        * memory.c: Likewise
        * objects.c: Likewise

boehm-gc/

2016-11-19  Matthias Klose  <[hidden email]>

        Remove




libobjc-bdw-gc.diff (15K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Sandra Loosemore
On 11/20/2016 01:42 PM, Matthias Klose wrote:

> +The options @option{--with-target-bdw-gc-include} and
> +@option{--with-target-bdw-gc-include} must always specified together for
> +each multilib variant and take precedence over
> +@option{--with-target-bdw-gc-include}.  If none of these options are
> +specified, the values are taken from the @command{pkg-config}
> +@samp{bdw-gc} module.

This paragraph makes no sense to me.  :-(

-Sandra

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Iain Sandoe-3
In reply to this post by Matthias Klose-6

> On 20 Nov 2016, at 20:42, Matthias Klose <[hidden email]> wrote:
>
> On 10.10.2016 09:58, Iain Sandoe wrote:
>>

>> The point here was to simplify the dependent configury so that it only needs to test something that the configuring user specifies (i.e. if they specify objc-gc, then they need also to specify the place that the gc lib can be found).
>
> So here is the next proposal, I hope the added documentation in install.texi
> makes the usage clear.

thanks for working on this!

>
> <toplevel>
>
> 2016-11-19  Matthias Klose  <[hidden email]>
>
> * Makefile.def: Remove reference to boehm-gc target module.
>   * configure.ac: Include pkg.m4, check for --with-target-bdw-gc
> options and for the bdw-gc pkg-config module.
> * configure: Regenerate.
> * Makefile.in: Regenerate.


+AC_ARG_ENABLE(objc-gc,
+[AS_HELP_STRING([--enable-objc-gc],
+ [enable use of Boehm's garbage collector with the
+ GNU Objective-C runtime])])
+AC_ARG_WITH([target-bdw-gc],
+[AS_HELP_STRING([--with-target-bdw-gc=PATHLIST],
+ [specify prefix directory for installed bdw-gc package.
+ Equivalent to --with-bdw-gc-include=PATH/include
+ plus --with-bdw-gc-lib=PATH/lib])])

missing “target” in the --with-bdw-gc-*

>
> gcc/
>
> 2016-11-19  Matthias Klose  <[hidden email]>
>
> * doc/install.texi: Document configure options --enable-objc-gc
> and --with-target-bdw-gc.

As per Sandra’s comment,  should we understand the priority of options is

--with-target-bdw-gc-*

which overrides…

--with-target-bdw-gc=<list>

which overrides automatic discovery using pkg_config?

Iain



Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Matthias Klose-6
On 21.11.2016 11:23, Iain Sandoe wrote:

>
>> On 20 Nov 2016, at 20:42, Matthias Klose <[hidden email]> wrote:
>>
>> On 10.10.2016 09:58, Iain Sandoe wrote:
>>>
>
>>> The point here was to simplify the dependent configury so that it only needs to test something that the configuring user specifies (i.e. if they specify objc-gc, then they need also to specify the place that the gc lib can be found).
>>
>> So here is the next proposal, I hope the added documentation in install.texi
>> makes the usage clear.
>
> thanks for working on this!
>
>>
>> <toplevel>
>>
>> 2016-11-19  Matthias Klose  <[hidden email]>
>>
>> * Makefile.def: Remove reference to boehm-gc target module.
>>   * configure.ac: Include pkg.m4, check for --with-target-bdw-gc
>> options and for the bdw-gc pkg-config module.
>> * configure: Regenerate.
>> * Makefile.in: Regenerate.
>
>
> +AC_ARG_ENABLE(objc-gc,
> +[AS_HELP_STRING([--enable-objc-gc],
> + [enable use of Boehm's garbage collector with the
> + GNU Objective-C runtime])])
> +AC_ARG_WITH([target-bdw-gc],
> +[AS_HELP_STRING([--with-target-bdw-gc=PATHLIST],
> + [specify prefix directory for installed bdw-gc package.
> + Equivalent to --with-bdw-gc-include=PATH/include
> + plus --with-bdw-gc-lib=PATH/lib])])
>
> missing “target” in the --with-bdw-gc-*

thanks, fixed.

>> gcc/
>>
>> 2016-11-19  Matthias Klose  <[hidden email]>
>>
>> * doc/install.texi: Document configure options --enable-objc-gc
>> and --with-target-bdw-gc.
>
> As per Sandra’s comment,  should we understand the priority of options is
>
> --with-target-bdw-gc-*
>
> which overrides…
>
> --with-target-bdw-gc=<list>
>
> which overrides automatic discovery using pkg_config?

--with-target-bdw-gc=/opt/bdw-gc,32=/opt/bdw-gc32

sets the include and lib dirs by appending include and lib to the paths. If you
have options --with-target-bdw-gc-include= and --with-target-bdw-gc-lib= as
well, it overrides the settings done in --with-target-bdw-gc=. This is copied
from the setting of the gmp/mpfr options.

Any of these options override the automatic discovery using pkg-config.

Please suggest a better wording; I thought that was clear enough (and better
than the undocumented --enable-libobjc-gc anyway ;)

Matthias

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Sandra Loosemore
On 11/21/2016 05:57 AM, Matthias Klose wrote:

>
> --with-target-bdw-gc=/opt/bdw-gc,32=/opt/bdw-gc32
>
> sets the include and lib dirs by appending include and lib to the paths. If you
> have options --with-target-bdw-gc-include= and --with-target-bdw-gc-lib= as
> well, it overrides the settings done in --with-target-bdw-gc=. This is copied
> from the setting of the gmp/mpfr options.
>
> Any of these options override the automatic discovery using pkg-config.
>
> Please suggest a better wording; I thought that was clear enough (and better
> than the undocumented --enable-libobjc-gc anyway ;)

FAOD, my complaint about your patch is that it essentially said

"The options --with-foo and --with-foo must also be specified together
and override --with-foo."

I'm sure you didn't mean exactly the same --with-foo in all 3 places,
but I'm a dummy about what you really intended to say here.

-Sandra


Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Matthias Klose-6
On 21.11.2016 17:23, Sandra Loosemore wrote:

> On 11/21/2016 05:57 AM, Matthias Klose wrote:
>>
>> --with-target-bdw-gc=/opt/bdw-gc,32=/opt/bdw-gc32
>>
>> sets the include and lib dirs by appending include and lib to the paths. If you
>> have options --with-target-bdw-gc-include= and --with-target-bdw-gc-lib= as
>> well, it overrides the settings done in --with-target-bdw-gc=. This is copied
>> from the setting of the gmp/mpfr options.
>>
>> Any of these options override the automatic discovery using pkg-config.
>>
>> Please suggest a better wording; I thought that was clear enough (and better
>> than the undocumented --enable-libobjc-gc anyway ;)
>
> FAOD, my complaint about your patch is that it essentially said
>
> "The options --with-foo and --with-foo must also be specified together and
> override --with-foo."
>
> I'm sure you didn't mean exactly the same --with-foo in all 3 places, but I'm a
> dummy about what you really intended to say here.

ahh, didn't see that :-/ Now fixed, is this clearer now?

The options @option{--with-target-bdw-gc-include} and
@option{--with-target-bdw-gc-lib} must always specified together for
each multilib variant and take precedence over
@option{--with-target-bdw-gc}.  If none of these options are
specified, the values are taken from the @command{pkg-config}
@samp{bdw-gc} module.

Matthias

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Peter Bergner-2
On 11/21/16 10:40 AM, Matthias Klose wrote:

> On 21.11.2016 17:23, Sandra Loosemore wrote:
>> On 11/21/2016 05:57 AM, Matthias Klose wrote:
>> I'm sure you didn't mean exactly the same --with-foo in all 3 places, but I'm a
>> dummy about what you really intended to say here.
>
> ahh, didn't see that :-/ Now fixed, is this clearer now?
>
> The options @option{--with-target-bdw-gc-include} and
> @option{--with-target-bdw-gc-lib} must always specified together for
> each multilib variant and take precedence over
> @option{--with-target-bdw-gc}.  If none of these options are
> specified, the values are taken from the @command{pkg-config}
> @samp{bdw-gc} module.

s/must always specified/must always be specified/

and possibly:

s/variant and take precedence/variant and they take precedence/ ???


Peter

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Rainer Orth-2
In reply to this post by Matthias Klose-6
Hi Matthias,

> ahh, didn't see that :-/ Now fixed, is this clearer now?
>
> The options @option{--with-target-bdw-gc-include} and
> @option{--with-target-bdw-gc-lib} must always specified together for
                                               ^ be
        Rainer

--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Matthias Klose-6
On 21.11.2016 18:16, Rainer Orth wrote:
> Hi Matthias,
>
>> ahh, didn't see that :-/ Now fixed, is this clearer now?
>>
>> The options @option{--with-target-bdw-gc-include} and
>> @option{--with-target-bdw-gc-lib} must always specified together for
>                                                ^ be

thanks to all sorting out the documentation issues. Now attaching the updated
diff. Ok to commit?

Matthias

<toplevel>

2016-11-19  Matthias Klose  <[hidden email]>

        * Makefile.def: Remove reference to boehm-gc target module.
        * configure.ac: Include pkg.m4, check for --with-target-bdw-gc
        options and for the bdw-gc pkg-config module.
        * configure: Regenerate.
        * Makefile.in: Regenerate.

gcc/

2016-11-19  Matthias Klose  <[hidden email]>

        * doc/install.texi: Document configure options --enable-objc-gc
        and --with-target-bdw-gc.

config/

2016-11-19  Matthias Klose  <[hidden email]>

        * pkg.m4: New file.

libobjc/

2016-11-19  Matthias Klose  <[hidden email]>

        * configure.ac (--enable-objc-gc): Allow to configure with a
        system provided boehm-gc.
        * configure: Regenerate.
        * Makefile.in (OBJC_BOEHM_GC_LIBS): Get value from configure.
        * gc.c: Include system bdw-gc headers.
        * memory.c: Likewise
        * objects.c: Likewise

boehm-gc/

2016-11-19  Matthias Klose  <[hidden email]>

        Remove


libobjc-bdw-gc.diff (49K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Sandra Loosemore
On 11/21/2016 04:23 PM, Matthias Klose wrote:

> On 21.11.2016 18:16, Rainer Orth wrote:
>> Hi Matthias,
>>
>>> ahh, didn't see that :-/ Now fixed, is this clearer now?
>>>
>>> The options @option{--with-target-bdw-gc-include} and
>>> @option{--with-target-bdw-gc-lib} must always specified together for
>>                                                 ^ be
>
> thanks to all sorting out the documentation issues. Now attaching the updated
> diff. Ok to commit?

The documentation part is OK now.

-Sandra

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Jeff Law
In reply to this post by Matthias Klose-6
On 11/21/2016 04:23 PM, Matthias Klose wrote:

> On 21.11.2016 18:16, Rainer Orth wrote:
>> Hi Matthias,
>>
>>> ahh, didn't see that :-/ Now fixed, is this clearer now?
>>>
>>> The options @option{--with-target-bdw-gc-include} and
>>> @option{--with-target-bdw-gc-lib} must always specified together for
>>                                                ^ be
>
> thanks to all sorting out the documentation issues. Now attaching the updated
> diff. Ok to commit?
>
> Matthias
>
> <toplevel>
>
> 2016-11-19  Matthias Klose  <[hidden email]>
>
>         * Makefile.def: Remove reference to boehm-gc target module.
>         * configure.ac: Include pkg.m4, check for --with-target-bdw-gc
>         options and for the bdw-gc pkg-config module.
>         * configure: Regenerate.
>         * Makefile.in: Regenerate.
>
> gcc/
>
> 2016-11-19  Matthias Klose  <[hidden email]>
>
>         * doc/install.texi: Document configure options --enable-objc-gc
>         and --with-target-bdw-gc.
>
> config/
>
> 2016-11-19  Matthias Klose  <[hidden email]>
>
>         * pkg.m4: New file.
>
> libobjc/
>
> 2016-11-19  Matthias Klose  <[hidden email]>
>
>         * configure.ac (--enable-objc-gc): Allow to configure with a
>         system provided boehm-gc.
>         * configure: Regenerate.
>         * Makefile.in (OBJC_BOEHM_GC_LIBS): Get value from configure.
>         * gc.c: Include system bdw-gc headers.
>         * memory.c: Likewise
>         * objects.c: Likewise
>
> boehm-gc/
>
> 2016-11-19  Matthias Klose  <[hidden email]>
>
>         Remove
OK.

Jeff
Reply | Threaded
Open this post in threaded view
|

Remove stray '@' from install.texi (was Re: [PATCH] Delete GCJ)

David Malcolm
On Tue, 2016-11-29 at 14:23 -0700, Jeff Law wrote:

> On 11/21/2016 04:23 PM, Matthias Klose wrote:
> > On 21.11.2016 18:16, Rainer Orth wrote:
> > > Hi Matthias,
> > >
> > > > ahh, didn't see that :-/ Now fixed, is this clearer now?
> > > >
> > > > The options @option{--with-target-bdw-gc-include} and
> > > > @option{--with-target-bdw-gc-lib} must always specified
> > > > together for
> > >                                                ^ be
> >
> > thanks to all sorting out the documentation issues. Now attaching
> > the updated
> > diff. Ok to commit?
> >
> > Matthias
> >
> >
[...]

> > gcc/
> >
> > 2016-11-19  Matthias Klose  <[hidden email]>
> >
> >         * doc/install.texi: Document configure options --enable
> > -objc-gc
> >         and --with-target-bdw-gc.

[...]

r242985 seems to have broken the build, for me at least (with texinfo
5.1):

../../src/gcc/doc/install.texi:2199: use braces to give a command as an argument to @=
make[2]: *** [doc/gccinstall.info] Error 1

The attached patch fixes it.

OK to commit?
Dave

0001-Remove-stray-character-from-install.texi.patch (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Remove stray '@' from install.texi (was Re: [PATCH] Delete GCJ)

Sandra Loosemore
On 11/29/2016 06:10 PM, David Malcolm wrote:

>[snip]
>
> r242985 seems to have broken the build, for me at least (with texinfo
> 5.1):
>
> ../../src/gcc/doc/install.texi:2199: use braces to give a command as an argument to @=
> make[2]: *** [doc/gccinstall.info] Error 1
>
> The attached patch fixes it.
>
> OK to commit?

OK.  (This is so trivial it would qualify under the obvious patch rule
anyway.)

-Sandra

Reply | Threaded
Open this post in threaded view
|

Re: Remove stray '@' from install.texi (was Re: [PATCH] Delete GCJ)

David Malcolm
On Tue, 2016-11-29 at 18:20 -0700, Sandra Loosemore wrote:

> On 11/29/2016 06:10 PM, David Malcolm wrote:
> > [snip]
> >
> > r242985 seems to have broken the build, for me at least (with
> > texinfo
> > 5.1):
> >
> > ../../src/gcc/doc/install.texi:2199: use braces to give a command
> > as an argument to @=
> > make[2]: *** [doc/gccinstall.info] Error 1
> >
> > The attached patch fixes it.
> >
> > OK to commit?
>
> OK.  (This is so trivial it would qualify under the obvious patch
> rule
> anyway.)
>
> -Sandra

My texinfo skills aren't as strong as yours, so I was half-wondering if
this was some syntax I wasn't aware of, or maybe a version issue.

Thanks for the confirmation; committed to trunk as r242991.

Dave
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Gerald Pfeifer
In reply to this post by Andrew Haley
On Fri, 30 Sep 2016, Andrew Haley wrote:
>> Note I did not include all the removed java/* contents.  Is there
>> anything particular you'd like to retain there?
> No, please delete it all.

Okay, so I (finally) went ahead and removed all of java/papers:

cvs commit: Examining java/papers
cvs commit: Examining java/papers/cni
Removing java/papers/README.sgml;
/cvs/gcc/wwwdocs/htdocs/java/papers/README.sgml,v  <--  README.sgml
new revision: delete; previous revision: 1.1
done
Removing java/papers/cni.sgml;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni.sgml,v  <--  cni.sgml
new revision: delete; previous revision: 1.5
done
Removing java/papers/compcon97.ps.gz;
/cvs/gcc/wwwdocs/htdocs/java/papers/compcon97.ps.gz,v  <--  compcon97.ps.gz
new revision: delete; previous revision: 1.1
done
Removing java/papers/compiling.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/compiling.html,v  <--  compiling.html
new revision: delete; previous revision: 1.3
done
Removing java/papers/compjava.pdf;
/cvs/gcc/wwwdocs/htdocs/java/papers/compjava.pdf,v  <--  compjava.pdf
new revision: delete; previous revision: 1.1
done
Removing java/papers/compjava.ps.gz;
/cvs/gcc/wwwdocs/htdocs/java/papers/compjava.ps.gz,v  <--  compjava.ps.gz
new revision: delete; previous revision: 1.1
done
Removing java/papers/esc97.sgml;
/cvs/gcc/wwwdocs/htdocs/java/papers/esc97.sgml,v  <--  esc97.sgml
new revision: delete; previous revision: 1.1
done
Removing java/papers/esc97w-slides.ps.gz;
/cvs/gcc/wwwdocs/htdocs/java/papers/esc97w-slides.ps.gz,v  <--  esc97w-slides.ps.gz
new revision: delete; previous revision: 1.1
done
Removing java/papers/gcc-java.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/gcc-java.html,v  <--  gcc-java.html
new revision: delete; previous revision: 1.6
done
Removing java/papers/gcc-java.ps.gz;
/cvs/gcc/wwwdocs/htdocs/java/papers/gcc-java.ps.gz,v  <--  gcc-java.ps.gz
new revision: delete; previous revision: 1.1
done
Removing java/papers/native++.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/native++.html,v  <--  native++.html
new revision: delete; previous revision: 1.3
done
Removing java/papers/native++.sgml;
/cvs/gcc/wwwdocs/htdocs/java/papers/native++.sgml,v  <--  native++.sgml
new revision: delete; previous revision: 1.3
done
Removing java/papers/nosb.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/nosb.html,v  <--  nosb.html
new revision: delete; previous revision: 1.6
done
Removing java/papers/nosb.ps.gz;
/cvs/gcc/wwwdocs/htdocs/java/papers/nosb.ps.gz,v  <--  nosb.ps.gz
new revision: delete; previous revision: 1.1
done
Removing java/papers/page.png;
/cvs/gcc/wwwdocs/htdocs/java/papers/page.png,v  <--  page.png
new revision: delete; previous revision: 1.1
done
Removing java/papers/table.png;
/cvs/gcc/wwwdocs/htdocs/java/papers/table.png,v  <--  table.png
new revision: delete; previous revision: 1.1
done
Removing java/papers/cni/HTML.manifest;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/HTML.manifest,v  <--  HTML.manifest
new revision: delete; previous revision: 1.3
done
Removing java/papers/cni/t1.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1.html,v  <--  t1.html
new revision: delete; previous revision: 1.3
done
Removing java/papers/cni/t1178.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1178.html,v  <--  t1178.html
new revision: delete; previous revision: 1.2
done
Removing java/papers/cni/t1215.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1215.html,v  <--  t1215.html
new revision: delete; previous revision: 1.2
done
Removing java/papers/cni/t1262.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1262.html,v  <--  t1262.html
new revision: delete; previous revision: 1.3
done
Removing java/papers/cni/t1308.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1308.html,v  <--  t1308.html
new revision: delete; previous revision: 1.2
done
Removing java/papers/cni/t1324.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1324.html,v  <--  t1324.html
new revision: delete; previous revision: 1.2
done
Removing java/papers/cni/t1331.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1331.html,v  <--  t1331.html
new revision: delete; previous revision: 1.2
done
Removing java/papers/cni/t134.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t134.html,v  <--  t134.html
new revision: delete; previous revision: 1.3
done
Removing java/papers/cni/t1405.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1405.html,v  <--  t1405.html
new revision: delete; previous revision: 1.2
done
Removing java/papers/cni/t1421.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1421.html,v  <--  t1421.html
new revision: delete; previous revision: 1.3
done
Removing java/papers/cni/t1431.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1431.html,v  <--  t1431.html
new revision: delete; previous revision: 1.2
done
Removing java/papers/cni/t1457.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1457.html,v  <--  t1457.html
new revision: delete; previous revision: 1.1
done
Removing java/papers/cni/t1467.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1467.html,v  <--  t1467.html
new revision: delete; previous revision: 1.2
done
Removing java/papers/cni/t187.html;
/cvs/gcc/wwwdocs/htdocs/java/papers/cni/t187.html,v  <--  t187.html
new revision: delete; previous revision: 1.2
done
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Per Bothner
On 01/22/2017 08:51 AM, Gerald Pfeifer wrote:
> On Fri, 30 Sep 2016, Andrew Haley wrote:
>>> Note I did not include all the removed java/* contents.  Is there
>>> anything particular you'd like to retain there?
>> No, please delete it all.
>
> Okay, so I (finally) went ahead and removed all of java/papers:

I don't think those should have been deleted.  There may be links to
them, and they may be historical interest.
--
        --Per Bothner
[hidden email]   http://per.bothner.com/
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Gerald Pfeifer
In reply to this post by Andrew Haley
And here is more or less the last patch in this series, removing
java/libgcj2.html and two images (gcj.jpg and swingshot.png).

The only thing left in wwwdocs is a potential merge of java/news.html
into our general news.html that I am considering.  That'd take a bit
more time, and I'll see whether I can do that before the GCC 7.1 release.

Gerald

Index: java/libgcj2.html
===================================================================
RCS file: java/libgcj2.html
diff -N java/libgcj2.html
--- java/libgcj2.html 30 Jun 2014 09:29:09 -0000 1.12
+++ /dev/null 1 Jan 1970 00:00:00 -0000
@@ -1,38 +0,0 @@
-<html>
-
-<head>
-<title>The libgcj home page</title>
-</head>
-
-<body>
-
-    <h1>The libgcj home page</h1>
-
-    <h2>What is it?</h2>
-    
-    <p>``libgcj'' is the runtime that goes along with the <a
-    href="gcj2.html">gcj</a> front end to GCC.  libgcj includes parts
-    of the Java Class Libraries, plus glue to connect the libraries to
-    the compiler and the underlying OS.</p>
-
-    <h2>What you get</h2>
-
-    <p>libgcj eventually builds a couple of libraries (one for the
-    runtime and one for the garbage collector), a ``zip'' version of
-    the class libraries, and a program called ``jv-convert'' which can
-    be used to do character encoding transformations.</p>
-
-    <h2>What is missing?</h2>
-
-    <p>The runtime is not yet fully complete.  Parts of the standard
-    class libraries are missing (most notably AWT), though much of the
-    equivalent of JDK 1.2 has been supported.
-    Also, the bytecode interpreter available only on certain platforms.</p>
-
-    <h2>How to build it</h2>
-
-    <p>Just follow our <a href="https://gcc.gnu.org/install/">installation
-    instructions</a>.</p>
-
-</body>
-</html>
Index: java/gcj.jpg
===================================================================
RCS file: java/gcj.jpg
diff -N java/gcj.jpg
Binary files /tmp/cvseTgpza and /dev/null differ
Index: java/swingshot.png
===================================================================
RCS file: java/swingshot.png
diff -N java/swingshot.png
Binary files /tmp/cvsGkz1jF and /dev/null differ
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Per Bothner
In reply to this post by Gerald Pfeifer
In my opinion, all/most of these should be restored.

On 01/22/2017 08:51 AM, Gerald Pfeifer wrote:

> On Fri, 30 Sep 2016, Andrew Haley wrote:
>>> Note I did not include all the removed java/* contents.  Is there
>>> anything particular you'd like to retain there?
>> No, please delete it all.
>
> Okay, so I (finally) went ahead and removed all of java/papers:
>
> cvs commit: Examining java/papers
> cvs commit: Examining java/papers/cni
> Removing java/papers/README.sgml;
> /cvs/gcc/wwwdocs/htdocs/java/papers/README.sgml,v  <--  README.sgml
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/cni.sgml;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni.sgml,v  <--  cni.sgml
> new revision: delete; previous revision: 1.5
> done
> Removing java/papers/compcon97.ps.gz;
> /cvs/gcc/wwwdocs/htdocs/java/papers/compcon97.ps.gz,v  <--  compcon97.ps.gz
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/compiling.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/compiling.html,v  <--  compiling.html
> new revision: delete; previous revision: 1.3
> done
> Removing java/papers/compjava.pdf;
> /cvs/gcc/wwwdocs/htdocs/java/papers/compjava.pdf,v  <--  compjava.pdf
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/compjava.ps.gz;
> /cvs/gcc/wwwdocs/htdocs/java/papers/compjava.ps.gz,v  <--  compjava.ps.gz
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/esc97.sgml;
> /cvs/gcc/wwwdocs/htdocs/java/papers/esc97.sgml,v  <--  esc97.sgml
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/esc97w-slides.ps.gz;
> /cvs/gcc/wwwdocs/htdocs/java/papers/esc97w-slides.ps.gz,v  <--  esc97w-slides.ps.gz
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/gcc-java.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/gcc-java.html,v  <--  gcc-java.html
> new revision: delete; previous revision: 1.6
> done
> Removing java/papers/gcc-java.ps.gz;
> /cvs/gcc/wwwdocs/htdocs/java/papers/gcc-java.ps.gz,v  <--  gcc-java.ps.gz
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/native++.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/native++.html,v  <--  native++.html
> new revision: delete; previous revision: 1.3
> done
> Removing java/papers/native++.sgml;
> /cvs/gcc/wwwdocs/htdocs/java/papers/native++.sgml,v  <--  native++.sgml
> new revision: delete; previous revision: 1.3
> done
> Removing java/papers/nosb.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/nosb.html,v  <--  nosb.html
> new revision: delete; previous revision: 1.6
> done
> Removing java/papers/nosb.ps.gz;
> /cvs/gcc/wwwdocs/htdocs/java/papers/nosb.ps.gz,v  <--  nosb.ps.gz
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/page.png;
> /cvs/gcc/wwwdocs/htdocs/java/papers/page.png,v  <--  page.png
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/table.png;
> /cvs/gcc/wwwdocs/htdocs/java/papers/table.png,v  <--  table.png
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/cni/HTML.manifest;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/HTML.manifest,v  <--  HTML.manifest
> new revision: delete; previous revision: 1.3
> done
> Removing java/papers/cni/t1.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1.html,v  <--  t1.html
> new revision: delete; previous revision: 1.3
> done
> Removing java/papers/cni/t1178.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1178.html,v  <--  t1178.html
> new revision: delete; previous revision: 1.2
> done
> Removing java/papers/cni/t1215.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1215.html,v  <--  t1215.html
> new revision: delete; previous revision: 1.2
> done
> Removing java/papers/cni/t1262.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1262.html,v  <--  t1262.html
> new revision: delete; previous revision: 1.3
> done
> Removing java/papers/cni/t1308.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1308.html,v  <--  t1308.html
> new revision: delete; previous revision: 1.2
> done
> Removing java/papers/cni/t1324.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1324.html,v  <--  t1324.html
> new revision: delete; previous revision: 1.2
> done
> Removing java/papers/cni/t1331.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1331.html,v  <--  t1331.html
> new revision: delete; previous revision: 1.2
> done
> Removing java/papers/cni/t134.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t134.html,v  <--  t134.html
> new revision: delete; previous revision: 1.3
> done
> Removing java/papers/cni/t1405.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1405.html,v  <--  t1405.html
> new revision: delete; previous revision: 1.2
> done
> Removing java/papers/cni/t1421.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1421.html,v  <--  t1421.html
> new revision: delete; previous revision: 1.3
> done
> Removing java/papers/cni/t1431.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1431.html,v  <--  t1431.html
> new revision: delete; previous revision: 1.2
> done
> Removing java/papers/cni/t1457.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1457.html,v  <--  t1457.html
> new revision: delete; previous revision: 1.1
> done
> Removing java/papers/cni/t1467.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t1467.html,v  <--  t1467.html
> new revision: delete; previous revision: 1.2
> done
> Removing java/papers/cni/t187.html;
> /cvs/gcc/wwwdocs/htdocs/java/papers/cni/t187.html,v  <--  t187.html
> new revision: delete; previous revision: 1.2
> done
>

--
        --Per Bothner
[hidden email]   http://per.bothner.com/
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Andrew Haley
On 22/01/17 18:41, Per Bothner wrote:
> In my opinion, all/most of these should be restored.

Because of the historical interest?  That's a good point, and perhaps
I was too hasty.  Sorry.

Andrew.

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Delete GCJ

Jakub Jelinek
On Mon, Jan 23, 2017 at 09:01:24AM +0000, Andrew Haley wrote:
> On 22/01/17 18:41, Per Bothner wrote:
> > In my opinion, all/most of these should be restored.
>
> Because of the historical interest?  That's a good point, and perhaps
> I was too hasty.  Sorry.

But then it should be probably moved somewhere where Gerald won't have to
spent time maintaining those pages (verification of URLs in there, updating
them when they are moved etc.).  Also to make it clear that GCJ is no longer
supported.

        Jakub
1234