On Mon, 28 Feb 2005 21:29:26 -0600, Tom 'spot' Callaway wrote:
Its either:
We use %{?disttag:.%{disttag}} in the Release: field, and we
conditionalize like this: %if "%{disttag}" == "2.fc3"
Or, we use %{?disttag:%{disttag}} in the Release: field, and we
conditionalize like this: %if "%{disttag}" == ".2.fc3"
I hope we never conditionalise like that, where we must look up weird dist
tags in a table, because we don't have an easier way how to determine the
actual build target platform conditionally.
If we go as far as evaluating %{disttag} instead of keeping it fully
optional -- %{?disttag:...} means it can be undefined (!) -- we better
define an unambiguous %{dist} or %{distribution} constant somewhere.
It doesn't matter to me. I'd prefer the extra period in the
Release
field with the simplified conditionals, but if everyone else prefers the
other way around, then I'll document the other way.
+1 to Enrico's view. A simple and clean %{?disttag} appended to the release
field. And since it can expand to virtually everything, don't abuse it for
conditionally _guessing_ build platforms.