After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 797335 - Building pkg-config files always happens with silent rules; doesn't respect --disable-silent-rules
Building pkg-config files always happens with silent rules; doesn't respect -...
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
1.14.4
Other Mac OS
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-10-25 11:41 UTC by Ryan Schmidt
Modified: 2018-11-03 12:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (325 bytes, patch)
2018-10-25 11:41 UTC, Ryan Schmidt
needs-work Details | Review

Description Ryan Schmidt 2018-10-25 11:41:06 UTC
Created attachment 374036 [details] [review]
patch

If I configure gstreamer 1.14.4 or 0.10.36 with --disable-silent-rules, silent rules are disabled, except for the part that "builds" (copies) the pkg-config .pc files; this part always displays e.g. "  CP     gstreamer-1.0.pc" instead of the actual cp command like I wanted it to.

I've attached what I believe to be the correct fix for gstreamer, but it looks like the problem affects the plugins as well and should be committed to each of them. I did not check all of your repositories so I don't know where all the fix is needed. I did see that gst-libav already includes this fix, and that some others like gst-editing-services and gst-rtsp-server appear (based on looking at the source code only) to have the opposite problem of always using non-silent rules. It would be good to get the pkgconfig/Makefile.am files of all of the repositories that use it to be consistent.
Comment 1 Sebastian Dröge (slomo) 2018-10-27 09:36:33 UTC
Thanks for the patch!

Please provide patches in "git format-patch" format, see https://gstreamer.freedesktop.org/documentation/contribute/index.html#how-to-submit-patches
Comment 2 Ryan Schmidt 2018-10-27 14:31:40 UTC
I really don't want to jump through any more hoops to contribute.... I've already contributed the necessary code change.
Comment 3 Tim-Philipp Müller 2018-10-27 14:42:22 UTC
This is so you get attributed properly as the author.
Comment 4 GStreamer system administrator 2018-11-03 12:49:05 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gstreamer/issues/322.