Prev: spamass-milter-0.3.1_9 leaving open zombie processes.
Next: gio-querymodules in devel/glib20
From: Christer Edwards on 10 May 2010 20:20
As a disclaimer, I don't have much experience maintaining ports. I've
altered a few for my own personal use, but never delved too deeply
into the "official" details of it all.

I have been working with the upstream maintainer of ccache to solve an
issue related to ccache and buildworld on amd64. I have built ports on
my local machine(s) to use his latest releases (3.0pre0 and 3.0pre1).
I would like to create an official port for the newer version once
this fix is in place, but I'm running into complaints from portlint
that I'm not sure how to address.

I am eager to learn and I'm happy to dedicate the time to put this
together, if someone doesn't mind offering advice on getting it to
pass portlint's critical eye.

The upstream download is available here: http://samba.org/ftp/ccache/
My (current) port is available for inspection and feedback here:
http://zelut.org/projects/misc/ccache-3.0pre1-port.tar.gz

It is based primarily on the current (v2.4) ccache port, minus the
patches (which don't seem to be needed). I haven't updated the
Maintainer information or any other details so far.

Looking forward to feedback,

--
Christer Edwards
_______________________________________________
freebsd-ports(a)freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscribe(a)freebsd.org"

From: Sahil Tandon on 10 May 2010 21:37
On Mon, 10 May 2010, Christer Edwards wrote:

> but I'm running into complaints from portlint that I'm not sure how to
> address.
>
> I am eager to learn and I'm happy to dedicate the time to put this
> together, if someone doesn't mind offering advice on getting it to
> pass portlint's critical eye.

% portlint -a
WARN: Makefile: only one MASTER_SITE configured. Consider adding additional mirrors.
FATAL: Makefile: PORTVERSION looks illegal. You should modify "3.0pre1".

The first one can be ignored if there are truly no other mirrors, but we
know there are. :) You could enumerate them or use MASTER_SITE_SAMBA;
the following is untested and might have unintended consequences, so
please review the appropriate Mk files and proceed with caution. You
might also think of a more succinct solution.

MASTER_SITES= ${MASTER_SITE_SAMBA:S|/samba||}
MASTER_SITE_SUBDIR= ${PORTNAME}

Regarding portlint's FATAL output, please review Package Naming
Conventions in the Handbook. Specifically, "If the software version has
strings like ``alpha'', ``beta'', ``rc'', or ``pre'', take the first
letter and put it immediately after a period." So you could try
sometihng like:

PORTVERSION= 3.0.p1
DISTNAME= ${PORTNAME}-${PORTVERSION:S/.p/pre/}

--
Sahil Tandon <sahil(a)FreeBSD.org>
_______________________________________________
freebsd-ports(a)freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscribe(a)freebsd.org"

From: Christer Edwards on 10 May 2010 21:46
On Mon, May 10, 2010 at 7:37 PM, Sahil Tandon <sahil(a)freebsd.org> wrote:
> Regarding portlint's FATAL output, please review Package Naming
> Conventions in the Handbook. Specifically, "If the software version has
> strings like ``alpha'', ``beta'', ``rc'', or ``pre'', take the first
> letter and put it immediately after a period."  So you could try
> sometihng like:
>
>  PORTVERSION=   3.0.p1
>  DISTNAME=      ${PORTNAME}-${PORTVERSION:S/.p/pre/}

I have been trying a number of variations as suggested in the
Handbook, but your suggestion worked well. 'portlint -a' now only
warns about there being only one mirror. I will work on resolving that
next.

Thank you,

--
Christer Edwards
_______________________________________________
freebsd-ports(a)freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscribe(a)freebsd.org"

From: Alberto Villa on 10 May 2010 22:08
On Tuesday 11 May 2010 03:37:01 Sahil Tandon wrote:
> Regarding portlint's FATAL output, please review Package Naming
> Conventions in the Handbook. Specifically, "If the software version has
> strings like ``alpha'', ``beta'', ``rc'', or ``pre'', take the first
> letter and put it immediately after a period." So you could try
> sometihng like:
>
> PORTVERSION= 3.0.p1
> DISTNAME= ${PORTNAME}-${PORTVERSION:S/.p/pre/}

DISTVERSION= 3.0pre1
is shorter and cleaner
--
Alberto Villa, FreeBSD committer <avilla(a)FreeBSD.org>
http://people.FreeBSD.org/~avilla

FORTUNE'S FUN FACTS TO KNOW AND TELL: #44
Zebra's are colored with dark stripes on a light background.
From: Adam Vande More on 10 May 2010 22:23
On Mon, May 10, 2010 at 9:08 PM, Alberto Villa <avilla(a)freebsd.org> wrote:

> > PORTVERSION= 3.0.p1
> > DISTNAME= ${PORTNAME}-${PORTVERSION:S/.p/pre/}
>
> DISTVERSION= 3.0pre1
> is shorter and cleaner
>

That's longer, and cleaner is debatable.

--
Adam Vande More
_______________________________________________
freebsd-ports(a)freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscribe(a)freebsd.org"

 |  Next  |  Last
Pages: 1 2 3
Prev: spamass-milter-0.3.1_9 leaving open zombie processes.
Next: gio-querymodules in devel/glib20