<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Manuel Amador (Rudd-O) wrote:
<blockquote cite="mid200903102234.50856.rudd-o@rudd-o.com" type="cite">
  <meta name="qrichtext" content="1">
  <style type="text/css">p, li { white-space: pre-wrap; }</style>Hello,
guys,<br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
I have fixed distutils (and setuptools remains working) with the
attached patch. Now, RPMs will be built according to the Fedora Package
Naming Guidelines:<br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
<a class="moz-txt-link-freetext" href="http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Non-Numeric_Version_in_Release">http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Non-Numeric_Version_in_Release</a><br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
which I understand to be the most useful reference in terms of naming
pre-release packages. This should work correctly in at least:<br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
- Fedora<br>
- RHEL<br>
- SUSE<br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
I urge you patch your python 2.4s and 2.5s and 2.6s and push this
update to distributions. I have done that myself at my own repository.<br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
Now we can enjoy one more reason to build RPMs (and eggs! ... according
to my workbench at <a class="moz-txt-link-freetext" href="http://yum.rudd-o.com/SCRIPTS/">http://yum.rudd-o.com/SCRIPTS/</a> -- feel free to pick
its brains) DIRECTLY from the cheese shop, especially if you're using
pip.<br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
Oh, I also have pip at my repo (cd ../RPMS/noarch in my workbench).<br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
See attached patch. I will log bugs where it corresponds too.<br>
-- <br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
Manuel Amador (Rudd-O) <a class="moz-txt-link-rfc2396E" href="mailto:rudd-o@rudd-o.com"><rudd-o@rudd-o.com></a><br>
Rudd-O.com - <a class="moz-txt-link-freetext" href="http://rudd-o.com/">http://rudd-o.com/</a><br>
GPG key ID 0xC8D28B92 at <a class="moz-txt-link-freetext" href="http://wwwkeys.pgp.net/">http://wwwkeys.pgp.net/</a><br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
Now playing, courtesy of Amarok: Aqua - Cartoon heroes<br>
Windows 95 is not a virus. Viruses actually do something.<br>
  <p style="margin: 0px; text-indent: 0px;"><br>
  </p>
</blockquote>
Hi Manuel,<br>
  You worked on my problem!  Great.<br>
  So today what we have been doing to deal with the pre-release and
lexical ordering problem involving pre-releases is this:<br>
    We impose a restriction on how the pre-release is identified.  So
for example if you intend to end up with a final version-release of
5.0.0-1 and you want to first put out some betas or release candidates
then we have to name them as, 5.0.0-0_beta1, or 5.0.0-0_rc1 and this is
so that the lexical ordering for RPM will be correct.  In other words
you must put the pre-release designation into the 'release' part of
VERSION-RELEASE.  What we had seen developers doing previously was to
name these as 5.0.0_beta1 or 5.0.0_rc1 (making the pre-release
designation part of the 'version' string)  which then did not work for
the lexical ordering of the final release of 5.0.0-1 because 5.0.0
(version) was not lexically superior to 5.0.0_rc1.  So we were able to
solve this problem without any code changes to distutils.  But this
also presented a challenge for the other distribution targets such as
'sdist' because they were totally unaware of this 'version-release'
combination and only knew about 'version'.  So as a workaround we were
doing this:<br>
# WORKAROUND<br>
# define both version AND release<br>
version='5.0.0'<br>
release='1'<br>
# combine them for all targets except 'bdist_rpm'<br>
if sys.argv[1] != 'bdist_rpm':<br>
    version = version+'-'+release<br>
So this wasn't perfect but it actually worked quite well and we could
get 'sdist' to work properly in conjunction with 'bdist_rpm'.  <br>
So now with your patch all the targets should be able to set and use
both 'version' and 'release' and we don't need our workaround and that
will be great.<br>
<br>
Regards,<br>
Gerry<br>
<br>
<br>
<br>
<br>
</body>
</html>