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 598248 - Properly detect pre-releases
Properly detect pre-releases
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: common
git master
Other Linux
: Normal normal
: 0.10.26
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks: 598369 598370
 
 
Reported: 2009-10-13 09:31 UTC by Edward Hervey
Modified: 2010-02-09 10:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
m4/as-version: Make AS_NANO set RELEASE if nano != 1 (1.62 KB, patch)
2009-10-14 06:42 UTC, Edward Hervey
none Details | Review
m4/gst-args: Properly set the package name for git/pre-release/release (1.05 KB, patch)
2009-10-14 06:42 UTC, Edward Hervey
none Details | Review

Description Edward Hervey 2009-10-13 09:31:23 UTC
currently -Werror is used for pre-releases.

The problem is that it's added when GST_CVS/GST_GIT is True... which is when the nano number is present.

GST_CVS/GST_GIT should only be True for nano == 1
Comment 1 Edward Hervey 2009-10-14 06:42:17 UTC
Created attachment 145398 [details] [review]
m4/as-version: Make AS_NANO set RELEASE if nano != 1

If code using AS_NANO wants finer grained control, they should
use the PACKAGE_VERSION_NANO.
Comment 2 Edward Hervey 2009-10-14 06:42:22 UTC
Created attachment 145399 [details] [review]
m4/gst-args: Properly set the package name for git/pre-release/release
Comment 3 Edward Hervey 2009-10-14 06:44:14 UTC
The two patches above properly detect pre-releases.

They will set GST_CVS/GST_GIT to FALSE if nano != 1
They will set GST_PACKAGE_NAME to the proper name ("git","prerelease","source release")
Comment 4 Sebastian Dröge (slomo) 2009-10-14 06:49:08 UTC
Looks good to me
Comment 5 Edward Hervey 2009-10-14 08:32:37 UTC
commit 85d153002980ca1341b34681c2172bd00080b4ed
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Wed Oct 14 08:40:11 2009 +0200

    m4/gst-args: Properly set the package name for git/pre-release/release
    
    https://bugzilla.gnome.org/show_bug.cgi?id=598248

commit 6ca472b7404c89a5533d435cc5a09ec3f52d55f2
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Wed Oct 14 08:38:47 2009 +0200

    m4/as-version: Make AS_NANO set RELEASE if nano != 1
    
    If code using AS_NANO wants finer grained control, they should
    use the PACKAGE_VERSION_NANO.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=598248