<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" content="1" /><style type="text/css">p, li { white-space: pre-wrap; }</style></head><body style=" font-family:'Tahoma'; font-size:12pt; font-weight:400; font-style:normal;">OK, Gerry, I have addressed your concerns.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>1) Note that the release number is still squelched in the case of RPMs compiled from SVN checkouts of distutils packages. This is so that alpha, beta or release candidate packages can upgrade SVN checkouts without extra intervention from the packager. Alpha, beta and rc packages do obey the release variable just fine. The whole intention here is to put a sensible release variable and a sensible version variable when no release variable was specified.<br>
2) I have used a relsep variable that you can tweak to use a dot or an elephant if your package policy wants that (note that the Mandrake policy says nothing about that).<br>
3) If anything non-numeric is specified in the release, then you as a packager are indicating that you do not want this automatic behavior and the modifications are automatically skipped. Note that the whole point of the patch is to AVOID this extra work of specifying the release for prerelease packages.<br>
4) The cases you spoke about are addressed.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>Attached is the patch. Here is a map table with the default release (which happens to be "1":<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>-----------------------<br>
[rudd-o@tobey ~]$ for a in `cat versions` ; do python bdist_rpm.py rewrite $a 1 ; done<br>
1.0-1 1.0-1<br>
1.1-1 1.1-1<br>
1.1a-1 1.1-0.1.a<br>
1.1a2-1 1.1-0.1.a2<br>
1.1b1-1 1.1-0.1.b1<br>
1.1dev-1 1.1-0.0.dev<br>
1.1svn-1 1.1-0.0.svn<br>
1.1dev-r56554-1 1.1-0.0.56554dev<br>
1.1svn-r400-1 1.1-0.0.400svn<br>
1.1svn-r401-1 1.1-0.0.401svn<br>
1.2svn-r34-1 1.2-0.0.34svn<br>
5.5rc2-1 5.5-0.1.rc2<br>
0.9-1 0.9-1<br>
0.9.8-1 0.9.8-1<br>
20080706-1 20080706-1<br>
A2005-1 A2005-1<br>
ABCDEF15-1 ABCDEF15-1<br>
abc45-1 abc45-1<br>
0.9alpha23-1 0.9-0.1.alpha23<br>
A2-1 A2-1<br>
--------------------<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>Now here is a map table with a release with a non-integer release:<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>----------------<br>
[rudd-o@tobey ~]$ for a in `cat versions` ; do python bdist_rpm.py rewrite $a 0_1 ; done<br>
1.0-0_1 1.0-0_1<br>
1.1-0_1 1.1-0_1<br>
1.1a-0_1 1.1a-0_1<br>
1.1a2-0_1 1.1a2-0_1<br>
1.1b1-0_1 1.1b1-0_1<br>
1.1dev-0_1 1.1dev-0_1<br>
1.1svn-0_1 1.1svn-0_1<br>
1.1dev-r56554-0_1 1.1dev_r56554-0_1<br>
1.1svn-r400-0_1 1.1svn_r400-0_1<br>
1.1svn-r401-0_1 1.1svn_r401-0_1<br>
1.2svn-r34-0_1 1.2svn_r34-0_1<br>
5.5rc2-0_1 5.5rc2-0_1<br>
0.9-0_1 0.9-0_1<br>
0.9.8-0_1 0.9.8-0_1<br>
20080706-0_1 20080706-0_1<br>
A2005-0_1 A2005-0_1<br>
ABCDEF15-0_1 ABCDEF15-0_1<br>
abc45-0_1 abc45-0_1<br>
0.9alpha23-0_1 0.9alpha23-0_1<br>
A2-0_1 A2-0_1<br>
------------------<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>I hope the next note you have about this patch is as constructive as the last one. The intention here is to solve a problem, not to bikeshed.<br>
-- <br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p> Manuel Amador (Rudd-O) <rudd-o@rudd-o.com><br>
Rudd-O.com - http://rudd-o.com/<br>
GPG key ID 0xC8D28B92 at http://wwwkeys.pgp.net/<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>If puns were deli meat, this would be the wurst.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p></body></html>