* [patch] [libffi] do not install libffi library, headers and documentation
@ 2013-02-12 12:30 Matthias Klose
2013-02-12 12:44 ` Richard Biener
0 siblings, 1 reply; 9+ messages in thread
From: Matthias Klose @ 2013-02-12 12:30 UTC (permalink / raw)
To: GCC Patches; +Cc: GCJ-patches, libffi-discuss
[-- Attachment #1: Type: text/plain, Size: 311 bytes --]
The libffi library, headers and documentation are still installed, although
libffi provides separate releases for a long time. So do not install these
anymore as part of a GCC install. Tested with a build and an install with go
and java enabled (both using libffi_convenience). Ok for the trunk?
Matthias
[-- Attachment #2: noinst.diff --]
[-- Type: text/x-diff, Size: 1775 bytes --]
2013-02-12 Matthias Klose <doko@ubuntu.com>
* Makefile.am: Do not install texinfo documentation and the
libffi library.
* include/Makefile.am: Do not install header files.
* man/Makefile.am: Do not install man pages.
* Makefile.in, include/Makefile.in, man/Makefile.in: Regenerate.
Index: Makefile.am
===================================================================
--- Makefile.am (Revision 195973)
+++ Makefile.am (Arbeitskopie)
@@ -49,7 +49,7 @@
# Defines info, dvi, pdf and html targets
MAKEINFOFLAGS = -I $(srcdir)/../gcc/doc/include
-info_TEXINFOS = doc/libffi.texi
+noinst_info_TEXINFOS = doc/libffi.texi
# AM_CONDITIONAL on configure option --generated-files-in-srcdir
if GENINSRC
@@ -130,8 +130,7 @@
MAKEOVERRIDES=
-toolexeclib_LTLIBRARIES = libffi.la
-noinst_LTLIBRARIES = libffi_convenience.la
+noinst_LTLIBRARIES = libffi.la libffi_convenience.la
libffi_la_SOURCES = src/prep_cif.c src/types.c \
src/raw_api.c src/java_raw_api.c src/closures.c
Index: include/Makefile.am
===================================================================
--- include/Makefile.am (Revision 195973)
+++ include/Makefile.am (Arbeitskopie)
@@ -9,4 +9,4 @@
gcc_version := $(shell cat $(top_srcdir)/../gcc/BASE-VER)
toollibffidir := $(libdir)/gcc/$(target_alias)/$(gcc_version)/include
-toollibffi_HEADERS = ffi.h ffitarget.h
+noinst_HEADERS = ffi.h ffitarget.h
Index: man/Makefile.am
===================================================================
--- man/Makefile.am (Revision 195973)
+++ man/Makefile.am (Arbeitskopie)
@@ -4,5 +4,5 @@
EXTRA_DIST = ffi.3 ffi_call.3 ffi_prep_cif.3 ffi_prep_cif_var.3
-man_MANS = ffi.3 ffi_call.3 ffi_prep_cif.3 ffi_prep_cif_var.3
+noinst_man_MANS = ffi.3 ffi_call.3 ffi_prep_cif.3 ffi_prep_cif_var.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [patch] [libffi] do not install libffi library, headers and documentation
2013-02-12 12:30 [patch] [libffi] do not install libffi library, headers and documentation Matthias Klose
@ 2013-02-12 12:44 ` Richard Biener
2013-02-12 12:46 ` Richard Biener
0 siblings, 1 reply; 9+ messages in thread
From: Richard Biener @ 2013-02-12 12:44 UTC (permalink / raw)
To: Matthias Klose; +Cc: GCC Patches, GCJ-patches, libffi-discuss
On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose <doko@ubuntu.com> wrote:
> The libffi library, headers and documentation are still installed, although
> libffi provides separate releases for a long time. So do not install these
> anymore as part of a GCC install. Tested with a build and an install with go
> and java enabled (both using libffi_convenience). Ok for the trunk?
openSUSE is using the GCC provided libffi, so no, this is not ok (not at this
stage anyway). Also proper not-installing libffi would work by disabling
the maybe-install-target-libffi at the toplevel, not changing libffi makfiles
(which are supposed to be imported from upstream, no?)
Richard.
> Matthias
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [patch] [libffi] do not install libffi library, headers and documentation
2013-02-12 12:44 ` Richard Biener
@ 2013-02-12 12:46 ` Richard Biener
2013-02-18 17:03 ` Matthias Klose
0 siblings, 1 reply; 9+ messages in thread
From: Richard Biener @ 2013-02-12 12:46 UTC (permalink / raw)
To: Matthias Klose; +Cc: GCC Patches, GCJ-patches, libffi-discuss
On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener
<richard.guenther@gmail.com> wrote:
> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose <doko@ubuntu.com> wrote:
>> The libffi library, headers and documentation are still installed, although
>> libffi provides separate releases for a long time. So do not install these
>> anymore as part of a GCC install. Tested with a build and an install with go
>> and java enabled (both using libffi_convenience). Ok for the trunk?
>
> openSUSE is using the GCC provided libffi, so no, this is not ok (not at this
> stage anyway). Also proper not-installing libffi would work by disabling
> the maybe-install-target-libffi at the toplevel, not changing libffi makfiles
> (which are supposed to be imported from upstream, no?)
Thus, add no_install= true; to the libffi target module
> Richard.
>
>> Matthias
>>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [patch] [libffi] do not install libffi library, headers and documentation
2013-02-12 12:46 ` Richard Biener
@ 2013-02-18 17:03 ` Matthias Klose
2013-02-19 9:13 ` Richard Biener
0 siblings, 1 reply; 9+ messages in thread
From: Matthias Klose @ 2013-02-18 17:03 UTC (permalink / raw)
To: Richard Biener; +Cc: GCC Patches, GCJ-patches, libffi-discuss
[-- Attachment #1: Type: text/plain, Size: 1044 bytes --]
Am 12.02.2013 13:45, schrieb Richard Biener:
> On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener
> <richard.guenther@gmail.com> wrote:
>> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose <doko@ubuntu.com> wrote:
>>> The libffi library, headers and documentation are still installed, although
>>> libffi provides separate releases for a long time. So do not install these
>>> anymore as part of a GCC install. Tested with a build and an install with go
>>> and java enabled (both using libffi_convenience). Ok for the trunk?
>>
>> openSUSE is using the GCC provided libffi, so no, this is not ok (not at this
>> stage anyway). Also proper not-installing libffi would work by disabling
>> the maybe-install-target-libffi at the toplevel, not changing libffi makfiles
>> (which are supposed to be imported from upstream, no?)
>
> Thus, add no_install= true; to the libffi target module
updated patch attached, checked with a make install that no ffi headers and
libraries are installed. If not ok for 4.8, ok for 4.9 when it opens?
Matthias
[-- Attachment #2: noinst.diff --]
[-- Type: text/x-diff, Size: 1720 bytes --]
* Makefile.def (target_modules): Don't install libffi.
* Makefile.in: Regenerate.
Index: Makefile.def
===================================================================
--- Makefile.def (Revision 196115)
+++ Makefile.def (Arbeitskopie)
@@ -138,7 +138,7 @@
missing=maintainer-clean; };
target_modules = { module= winsup; };
target_modules = { module= libgloss; no_check=true; };
-target_modules = { module= libffi; };
+target_modules = { module= libffi; no_install=true; };
target_modules = { module= libjava; raw_cxx=true;
extra_configure_flags="$(EXTRA_CONFIGARGS_LIBJAVA)"; };
target_modules = { module= zlib; };
Index: Makefile.in
===================================================================
--- Makefile.in (Revision 196115)
+++ Makefile.in (Arbeitskopie)
@@ -38710,13 +38710,8 @@
@if target-libffi
maybe-install-target-libffi: install-target-libffi
-install-target-libffi: installdirs
- @: $(MAKE); $(unstage)
- @r=`${PWD_COMMAND}`; export r; \
- s=`cd $(srcdir); ${PWD_COMMAND}`; export s; \
- $(NORMAL_TARGET_EXPORTS) \
- (cd $(TARGET_SUBDIR)/libffi && \
- $(MAKE) $(TARGET_FLAGS_TO_PASS) install)
+# Dummy target for uninstallable.
+install-target-libffi:
@endif target-libffi
@@ -38725,13 +38720,8 @@
@if target-libffi
maybe-install-strip-target-libffi: install-strip-target-libffi
-install-strip-target-libffi: installdirs
- @: $(MAKE); $(unstage)
- @r=`${PWD_COMMAND}`; export r; \
- s=`cd $(srcdir); ${PWD_COMMAND}`; export s; \
- $(NORMAL_TARGET_EXPORTS) \
- (cd $(TARGET_SUBDIR)/libffi && \
- $(MAKE) $(TARGET_FLAGS_TO_PASS) install-strip)
+# Dummy target for uninstallable.
+install-strip-target-libffi:
@endif target-libffi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [patch] [libffi] do not install libffi library, headers and documentation
2013-02-18 17:03 ` Matthias Klose
@ 2013-02-19 9:13 ` Richard Biener
2013-03-26 20:04 ` [ping] " Matthias Klose
0 siblings, 1 reply; 9+ messages in thread
From: Richard Biener @ 2013-02-19 9:13 UTC (permalink / raw)
To: Matthias Klose; +Cc: GCC Patches, GCJ-patches, libffi-discuss
On Mon, Feb 18, 2013 at 6:02 PM, Matthias Klose <doko@ubuntu.com> wrote:
> Am 12.02.2013 13:45, schrieb Richard Biener:
>> On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener
>> <richard.guenther@gmail.com> wrote:
>>> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose <doko@ubuntu.com> wrote:
>>>> The libffi library, headers and documentation are still installed, although
>>>> libffi provides separate releases for a long time. So do not install these
>>>> anymore as part of a GCC install. Tested with a build and an install with go
>>>> and java enabled (both using libffi_convenience). Ok for the trunk?
>>>
>>> openSUSE is using the GCC provided libffi, so no, this is not ok (not at this
>>> stage anyway). Also proper not-installing libffi would work by disabling
>>> the maybe-install-target-libffi at the toplevel, not changing libffi makfiles
>>> (which are supposed to be imported from upstream, no?)
>>
>> Thus, add no_install= true; to the libffi target module
>
> updated patch attached, checked with a make install that no ffi headers and
> libraries are installed. If not ok for 4.8, ok for 4.9 when it opens?
I'm fine with that variant but I'd like to see another ok. No preference as to
whether to target 4.8 or 4.9.
Richard.
> Matthias
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [ping] Re: [patch] [libffi] do not install libffi library, headers and documentation
2013-02-19 9:13 ` Richard Biener
@ 2013-03-26 20:04 ` Matthias Klose
2013-03-26 20:28 ` Ian Lance Taylor
0 siblings, 1 reply; 9+ messages in thread
From: Matthias Klose @ 2013-03-26 20:04 UTC (permalink / raw)
To: Andrew Haley, Tom Tromey, Ian Lance Taylor
Cc: GCC Patches, GCJ-patches, libffi-discuss
[ping, adding the GCJ and Go maintainers]
proposed patch at http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00853.html
Am 19.02.2013 10:13, schrieb Richard Biener:
> On Mon, Feb 18, 2013 at 6:02 PM, Matthias Klose <doko@ubuntu.com> wrote:
>> Am 12.02.2013 13:45, schrieb Richard Biener:
>>> On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener
>>> <richard.guenther@gmail.com> wrote:
>>>> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose <doko@ubuntu.com> wrote:
>>>>> The libffi library, headers and documentation are still installed, although
>>>>> libffi provides separate releases for a long time. So do not install these
>>>>> anymore as part of a GCC install. Tested with a build and an install with go
>>>>> and java enabled (both using libffi_convenience). Ok for the trunk?
>>>>
>>>> openSUSE is using the GCC provided libffi, so no, this is not ok (not at this
>>>> stage anyway). Also proper not-installing libffi would work by disabling
>>>> the maybe-install-target-libffi at the toplevel, not changing libffi makfiles
>>>> (which are supposed to be imported from upstream, no?)
>>>
>>> Thus, add no_install= true; to the libffi target module
>>
>> updated patch attached, checked with a make install that no ffi headers and
>> libraries are installed. If not ok for 4.8, ok for 4.9 when it opens?
>
> I'm fine with that variant but I'd like to see another ok. No preference as to
> whether to target 4.8 or 4.9.
>
> Richard.
>
>> Matthias
>>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [ping] Re: [patch] [libffi] do not install libffi library, headers and documentation
2013-03-26 20:04 ` [ping] " Matthias Klose
@ 2013-03-26 20:28 ` Ian Lance Taylor
2013-03-26 20:48 ` Anthony Green
0 siblings, 1 reply; 9+ messages in thread
From: Ian Lance Taylor @ 2013-03-26 20:28 UTC (permalink / raw)
To: Matthias Klose
Cc: Andrew Haley, Tom Tromey, GCC Patches, GCJ-patches, libffi-discuss
On Tue, Mar 26, 2013 at 1:03 PM, Matthias Klose <doko@ubuntu.com> wrote:
> [ping, adding the GCJ and Go maintainers]
>
> proposed patch at http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00853.html
As far as I know this won't affect Go. So it's fine with me. But I'd
rather see this approved by a libffi maintainer. But there is no
libffi maintainer listed in MAINTAINERS. Hmmm.
Ian
> Am 19.02.2013 10:13, schrieb Richard Biener:
>> On Mon, Feb 18, 2013 at 6:02 PM, Matthias Klose <doko@ubuntu.com> wrote:
>>> Am 12.02.2013 13:45, schrieb Richard Biener:
>>>> On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener
>>>> <richard.guenther@gmail.com> wrote:
>>>>> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose <doko@ubuntu.com> wrote:
>>>>>> The libffi library, headers and documentation are still installed, although
>>>>>> libffi provides separate releases for a long time. So do not install these
>>>>>> anymore as part of a GCC install. Tested with a build and an install with go
>>>>>> and java enabled (both using libffi_convenience). Ok for the trunk?
>>>>>
>>>>> openSUSE is using the GCC provided libffi, so no, this is not ok (not at this
>>>>> stage anyway). Also proper not-installing libffi would work by disabling
>>>>> the maybe-install-target-libffi at the toplevel, not changing libffi makfiles
>>>>> (which are supposed to be imported from upstream, no?)
>>>>
>>>> Thus, add no_install= true; to the libffi target module
>>>
>>> updated patch attached, checked with a make install that no ffi headers and
>>> libraries are installed. If not ok for 4.8, ok for 4.9 when it opens?
>>
>> I'm fine with that variant but I'd like to see another ok. No preference as to
>> whether to target 4.8 or 4.9.
>>
>> Richard.
>>
>>> Matthias
>>>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [ping] Re: [patch] [libffi] do not install libffi library, headers and documentation
2013-03-26 20:28 ` Ian Lance Taylor
@ 2013-03-26 20:48 ` Anthony Green
2013-03-30 11:26 ` Matthias Klose
0 siblings, 1 reply; 9+ messages in thread
From: Anthony Green @ 2013-03-26 20:48 UTC (permalink / raw)
To: Ian Lance Taylor
Cc: Matthias Klose, Andrew Haley, Tom Tromey, GCC Patches,
GCJ-patches, libffi-discuss
For what it's worth, this patch is fine by me. I had originally
proposed that GCC not install these bits.
As far as maintainers go, I thought that I was once listed in the
MAINTAINERS file. Feel free to add Andrew Haley and/or myself.
Thanks,
Anthony Green
On Tue, Mar 26, 2013 at 4:28 PM, Ian Lance Taylor <iant@google.com> wrote:
> On Tue, Mar 26, 2013 at 1:03 PM, Matthias Klose <doko@ubuntu.com> wrote:
>> [ping, adding the GCJ and Go maintainers]
>>
>> proposed patch at http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00853.html
>
> As far as I know this won't affect Go. So it's fine with me. But I'd
> rather see this approved by a libffi maintainer. But there is no
> libffi maintainer listed in MAINTAINERS. Hmmm.
>
> Ian
>
>
>> Am 19.02.2013 10:13, schrieb Richard Biener:
>>> On Mon, Feb 18, 2013 at 6:02 PM, Matthias Klose <doko@ubuntu.com> wrote:
>>>> Am 12.02.2013 13:45, schrieb Richard Biener:
>>>>> On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener
>>>>> <richard.guenther@gmail.com> wrote:
>>>>>> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose <doko@ubuntu.com> wrote:
>>>>>>> The libffi library, headers and documentation are still installed, although
>>>>>>> libffi provides separate releases for a long time. So do not install these
>>>>>>> anymore as part of a GCC install. Tested with a build and an install with go
>>>>>>> and java enabled (both using libffi_convenience). Ok for the trunk?
>>>>>>
>>>>>> openSUSE is using the GCC provided libffi, so no, this is not ok (not at this
>>>>>> stage anyway). Also proper not-installing libffi would work by disabling
>>>>>> the maybe-install-target-libffi at the toplevel, not changing libffi makfiles
>>>>>> (which are supposed to be imported from upstream, no?)
>>>>>
>>>>> Thus, add no_install= true; to the libffi target module
>>>>
>>>> updated patch attached, checked with a make install that no ffi headers and
>>>> libraries are installed. If not ok for 4.8, ok for 4.9 when it opens?
>>>
>>> I'm fine with that variant but I'd like to see another ok. No preference as to
>>> whether to target 4.8 or 4.9.
>>>
>>> Richard.
>>>
>>>> Matthias
>>>>
>>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [ping] Re: [patch] [libffi] do not install libffi library, headers and documentation
2013-03-26 20:48 ` Anthony Green
@ 2013-03-30 11:26 ` Matthias Klose
0 siblings, 0 replies; 9+ messages in thread
From: Matthias Klose @ 2013-03-30 11:26 UTC (permalink / raw)
To: Anthony Green
Cc: Ian Lance Taylor, Andrew Haley, Tom Tromey, GCC Patches,
GCJ-patches, libffi-discuss
Am 26.03.2013 21:48, schrieb Anthony Green:
> For what it's worth, this patch is fine by me. I had originally
> proposed that GCC not install these bits.
now applied on trunk and the 4.8 branch.
> As far as maintainers go, I thought that I was once listed in the
> MAINTAINERS file. Feel free to add Andrew Haley and/or myself.
I didn't hear back form Andrew. Could you do this yourself, maybe updating
libffi on trunk to the 3.0.13 release? ;)
Matthias
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2013-03-30 11:26 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-12 12:30 [patch] [libffi] do not install libffi library, headers and documentation Matthias Klose
2013-02-12 12:44 ` Richard Biener
2013-02-12 12:46 ` Richard Biener
2013-02-18 17:03 ` Matthias Klose
2013-02-19 9:13 ` Richard Biener
2013-03-26 20:04 ` [ping] " Matthias Klose
2013-03-26 20:28 ` Ian Lance Taylor
2013-03-26 20:48 ` Anthony Green
2013-03-30 11:26 ` Matthias Klose
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).