Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 14 Oct 2010 20:24:59 -0400
From:      Steve Wills <steve@mouf.net>
To:        pgollucci@FreeBSD.org
Cc:        cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: ports/devel/rubygem-pkg-config pkg-descr
Message-ID:  <4CB79F5B.1060703@mouf.net>
In-Reply-To: <20101014153924.b290bd11.stas@FreeBSD.org>
References:  <201010122319.o9CNJD1A046613@repoman.freebsd.org>	<20101014131456.4d49bd3d.stas@FreeBSD.org>	<20101014215117.GB34544@atarininja.org> <20101014153924.b290bd11.stas@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Here's a draft of a reply... I'd appreciate comments before I send it...

Thanks,
Steve


On 10/14/10 18:39, Stanislav Sedov wrote:
> On Thu, 14 Oct 2010 17:51:17 -0400
> Wesley Shields <wxs@FreeBSD.org> mentioned:
> 
> 
> But you didn't heard that you should use 'co-mentor' either?

Actually, I did, from my mentor. I can't find the mail right now to
forward, but I'm pretty sure I didn't imagine it.`

> It was meant for consistency, so you can later do some automatic processing
> based on "(mentor)".  I don't even know where this string came from: you
> probably can't find any references to "approved by co-mentor" neither in
> the committer's guide, or in porter's handbook.
> 
> I just was surpised to receive no reaction on my first comment both from the
> committer and the mentor: the "co-mentor" nonsense has continued to be used.
> Maybe I just missed that, but I didn't heard that we allow each new mentor
> to invent his own string to put into parenthesis; if it's true, my I'll
> ask my next mentee to put something like "master" or "god" in place of
> "mentor", just because I like it better, or nobody told me that I shouldn't
> do that.
> 

As far as the other points go:

1. I fixed the WWW link as you saw.

2. The package docs and Rakefile say it depends on hoe. Is removing the
dependency necessary?

3. There are 3328 ports or so in ports/devel, of those 1352 or so seem
to have multiple hyphens in them, including things such as
"py-repoze.what-pylons", "rpm-spec-mode.el" (both with 2 hyphens and a
dot), "p5-Test-Fixture-DBIC-Schema" (4 hyphens), "rubygem-file-tail",
"rubygem-ruby-debug-ide" (both 2 or more hyphons and also rubygems). So
I disagree that "we usually don't use hypens in the portname".

Also, the port for pkg-config is called "pkg-config". If I were
searching for pkg-config related ports, I would search for "pkg-config",
not "pkgconfig". Changing it seems unnecessary and makes life harder for
others, IMHO.

I ran portlint -Ac (v2.13.1) and it didn't produce a warning for this:

% portlint -Ac
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section
empty, to make CVS happy.
FATAL: CVS: for safety, be sure to cleanup CVS directories before
importing the new port.
1 fatal error and 1 warning found.
%

(the CVS related warnings are of course not applicable because this has
already been committed)

4. Style issue which I'm willing to accept patches for, but don't have
time to work on at the moment.

5. Style issue which I'm willing to accept patches for, but don't have
time to work on at the moment.

Steve
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.16 (FreeBSD)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJMt59bAAoJEPXPYrMgexuh5uAH/RO2LUaub7anCsoxX8dgxdCI
gseG8svnfRm01EE/peFpZCbDkyScZ+7Ay0beg+/Oesx4XC5umOs+lcYHn76/k/eg
HJ8dXK5jtnUE5SUSNKOleIuZ+Hw6OosU0QTbvX41ZBTYRd/E7die8lAkNqvh6daJ
oPXrWKEjZ6dgUsgoXk8U5C+qutTV6qzBHslFbSTLY+itgvbZTyXesXl9xlSVB3Wj
SVpknVjTMNEub8UBzPBtLococunt4HMriBMuQ2PDkbXi8Niyi5pis2oy5tglerGu
CBV21oFkR/43Vuq0pcu2ZdXp+C8B3SbhzkiIk+0VsAdOH/hEnvGBR8wtRSYcHAQ=
=sfLL
-----END PGP SIGNATURE-----



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4CB79F5B.1060703>