GNOME Bugzilla – Bug 674898
Deal with GLIB_VERSION_MIN_REQUIRED/MAX_ALLOWED being a "future" value
Last modified: 2012-07-06 16:14:30 UTC
Undefined symbols behave as 0 in preprocessor math, so, eg, if you try to compile against glib 2.32 with -DGLIB_VERSION_MAX_ALLOWED=GLIB_VERSION_2_34, it essentially thinks you're saying MAX_ALLOWED is 0.0, and complains that that's less than MIN_REQUIRED. This fixes both MIN_REQUIRED and MAX_ALLOWED to treat all unrecognized version symbols as being in the future. I've gone back and forth a few times on whether it should bail out if it doesn't recognize MIN_REQUIRED, but eventually decided that that's configure's job...
Created attachment 212903 [details] [review] Deal with GLIB_VERSION_MIN_REQUIRED/MAX_ALLOWED being a "future" value If GLIB_VERSION_MIN_REQUIRED or GLIB_VERSION_MAX_ALLOWED was defined to a future value, we were essentially treating it as GLIB_VERSION_0_0. Fix to treat it as being in the future instead.
Created attachment 212904 [details] [review] Clarify the GLIB_VERSION_MIN_REQUIRED/MAX_ALLOWED docs
Review of attachment 212903 [details] [review]: ok
Review of attachment 212904 [details] [review]: ok
Patches don't apply cleanly anymore - Dan, can you rebase and push ?
Attachment 212903 [details] pushed as 40f0f66 - Deal with GLIB_VERSION_MIN_REQUIRED/MAX_ALLOWED being a "future" value Attachment 212904 [details] pushed as ee9aae5 - Clarify the GLIB_VERSION_MIN_REQUIRED/MAX_ALLOWED docs