cvs commit: ports/ftp/curl Makefile distinfo pkg-plist ports/ftp/curl/files patch-Makefile.in patch-configure patch-docs::TODO patch-docs::curl.1 patch-docs::curl.html patch-docs::examples::fileupload.c patch-docs::libcurl::curl_easy_pause.3 patch-docs::libcurl::curl_easy_pause.html ...

[Peter Pentchev]

On Wed, Apr 23, 2008 at 01:16:12PM +0000, Alexey Dokuchaev wrote:
> On Wed, Apr 23, 2008 at 11:33:34AM +0000, Peter Pentchev wrote:
> > roam        2008-04-23 11:33:34 UTC
> > 
> >   FreeBSD ports repository
> > 
> >   Modified files:
> >     ftp/curl             Makefile distinfo pkg-plist 
> >     ftp/curl/files       patch-configure 
> >   Added files:
> >     ftp/curl/files       patch-Makefile.in patch-docs::TODO 
> >                          patch-docs::curl.1 patch-docs::curl.html 
> >                          patch-docs::examples::fileupload.c 
> >                          patch-docs::libcurl::curl_easy_pause.3 
> >                          patch-docs::libcurl::curl_easy_pause.html 
> >                          patch-docs::libcurl::curl_multi_setopt.3 
> >                          patch-docs::libcurl::curl_multi_setopt.html 
> >                          patch-lib::easy.c patch-lib::file.c 
> >                          patch-lib::multi.c patch-src::main.c 
> >                          patch-tests::libtest::lib505.c 
> >                          patch-tests::libtest::lib541.c 
> >   Removed files:
> >     ftp/curl/files       patch-lib::ssh.c 
> >   Log:
> >   Update to curl-7.18.0.
> >   
> >   Revision  Changes    Path
> >   1.92      +45 -8     ports/ftp/curl/Makefile
> >   1.50      +3 -3      ports/ftp/curl/distinfo
> >   1.1       +29 -0     ports/ftp/curl/files/patch-Makefile.in (new)
> >   1.9       +14 -4     ports/ftp/curl/files/patch-configure
> >   1.1       +80 -0     ports/ftp/curl/files/patch-docs::TODO (new)
> >   1.1       +740 -0    ports/ftp/curl/files/patch-docs::curl.1 (new)
> >   1.1       +436 -0    ports/ftp/curl/files/patch-docs::curl.html (new)
> 
> 	[ .. log remainder skipped .. ]
> 
> We generally try to avoid using :: in patch names; per PH: "Please only
> use characters [-+._a-zA-Z0-9] for naming your patches."

I know, and I'm using a dash in a couple of other ports - security/mhash,
net/p5-WWW-Curl, mail/vpopmail, etc.

However, with the cURL port it's mainly a matter of consistency and
preserving the status quo at the price of ignoring the portlint warnings :)
I moved from the patch-aa to the :: convention several years ago, and there
are quite a lot of patches that would need repo-copying should I decide
to use a dash instead.

Still, thanks for the gentle reminder - in general :)

G'luck,
Peter

-- 
Peter Pentchev	roam at ringlet.net    roam at cnsys.bg    roam at FreeBSD.org
PGP key:	http://people.FreeBSD.org/~roam/roam.key.asc
Key fingerprint	FDBA FD79 C26F 3C51 C95E  DF9E ED18 B68D 1619 4553
because I didn't think of a good beginning of it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-ports/attachments/20080423/27e82c0e/attachment.pgp