Felipe Contreras <felipe.contre...@gmail.com> writes:

> If the version is 'v1.8.4-rc1' that is the version, and there's no need
> to change it to anything else, like 'v1.8.4.rc1'.
> Signed-off-by: Felipe Contreras <felipe.contre...@gmail.com>
> ---
>  1 file changed, 1 deletion(-)
> index b0db139..2b9fd2f 100755
> @@ -24,7 +24,6 @@ then
>               VN="$VN-dirty"
>               ;;
>       esac
> -     VN=$(echo "$VN" | sed -e 's/-/./g')
>  else
>       VN="$DEF_VER"
>  fi

The log message of 5c7d3c95 (Allow building of RPM from interim
snapshot., 2006-01-16) leaves a lot to be desired, but I do remember
that this was added due to user request.  Before that commit, we did
not strip dashes, and RPM packaging did not like dash in the version
name, hence folks on RH and derived distributions could not package
the software out of the box themselves.

I do not offhand know if that RPM limitation was lifted.  I do not
know if there are other packagers with a similar limitation that
have been relying on this version mangling, either.  This change may
end up being a regression for these users.

The mangling is done only for auto-generated names, so I suspect
that anybody who wanted the dash in their version names for whatever
reason has already been using the "version file" escape hatch (or
nobody had a desparate need to use dash instead of dot).

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to