[beports] Re: CVS/SVN head portlog/patches

Hello,
I disagree in that respect. Naming the file cvs-CVS.diff (or cvs- HEAD.diff) allows it to be easily overwritten and revisioned with future patches inside our repository. In case of ccvs-CVS I don't think it'll change much anyway (last in 2005?). CVS repositories in general have the problem of not providing a global revision number, we would have to use the UTC date+time of checkout. This currently affects CVS, expat and Portable.NET.
For SVN I intended to name them "port-trunk" or according to their branches (e.g. subversion-1.4.x as opposed to subversion-1.4.6) since I update my repositories often and don't want to rename patch file and Wiki headings all the time. The revision number alone, while unique, does not reflect what it is against. And note that Trac will recognize rxxx revision numbers and interpret them as revisions of the BePorts repository, thus in the Wiki we would have to name them rev.xxx or something.
Correctly created patches against working copies (cvs diff -u, svn diff, git diff) contain the revision information in textual form at the beginning of the file, before each actual file diff.
I assume that we won't attempt to use the BePorter tool to use them in an automated way, they are solely intended to share my work in progress with you or anyone interested.
In the Wiki we might document one of the revisions used as a vague reference (i.e. without strict guarantees of being up-to-date) or for CVS add a statement "last updated:", but it wouldn't make sense to duplicate Wiki sections for each SVN revision as done for the released versions. The point should be to get ports done and not get us stopped by unneccessary documentation. It's been already some work to apply the template all over and to create. There's little point in providing outdated patches as part of a checkout (that's what our revisioned repository is for), and I also didn't see a reason to keep Wiki content for outdated Haiku revisions (e.g. sys/poll.h and sys/mman.h were added). Keeping info on old released software versions I understand and agree with. 

Regards,

Andreas

Am 17.04.2008 um 11:30 schrieb Brecht Machiels:


Hello,
I noticed Andreas provided a patch against the CVS head of dev-util/ cvs. 
As the CVS head sources will change over time, I suggest we name the
patches so that they indicate the CVS revision. I'm not sure how it is
with CVS, but for SVN repositories, it would be something like
port-r24713.diff. This way, even when the CVS head has changed
considerably, others can check out the revision in question and make
sense of the patch.
Similarly, the PortLog should also mention the cvs/svn revision the log 
corresponds to. So instead of "CVS HEAD", say "CVS revision ..."
In the weekend I will probably have time to make the appropriate changes 
to the wiki pages.

Best regards,
Brecht


--
BePorts homepage - http://tools.assembla.com/BePorts
List archives: http://www.freelists.org/archives/beports
Administrative contact: brecht@xxxxxxxxxxx



--
BePorts homepage - http://tools.assembla.com/BePorts
List archives: http://www.freelists.org/archives/beports
Administrative contact: brecht@xxxxxxxxxxx

Other related posts: