Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 14 Jul 2014 16:36:21 +0200
From:      Carlos Jacobo Puga Medina <cjpugmed@gmail.com>
To:        koobs@freebsd.org
Cc:        freebsd-ports@freebsd.org
Subject:   Re: Please commit bug 190843
Message-ID:  <CAHHLbRNg9wyqO1Ni6KqTTi1DtcFDbPBATpGokUyA13GSfuVsRQ@mail.gmail.com>
In-Reply-To: <53C3DD16.2040009@FreeBSD.org>
References:  <CAHHLbRPWk6a0p8NQ6RyFzY7j_1VYEzWZMR2mGNF3dDidZz5Euw@mail.gmail.com> <53C3DD16.2040009@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
2014-07-14 15:37 GMT+02:00 Kubilay Kocak <koobs@freebsd.org>:

> On 14/07/2014 11:23 PM, Carlos Jacobo Puga Medina wrote:
> > Added patch to update to 0.7.0
> >
> > Build logs:
> >
> > https://redports.org/buildarchive/20140714113400-20929/
> >
> > --CJPM
> > _______________________________________________
> > freebsd-ports@freebsd.org mailing list
> > http://lists.freebsd.org/mailman/listinfo/freebsd-ports
> > To unsubscribe, send any mail to "freebsd-ports-unsubscribe@freebsd.org"
> >
>
> Thanks for your submission Carlos :)
>
> Our issue tracker is the authoritative source of information for change
> requests, so always include all information there before posting on the
> mailing lists.
>
> I also note that OPTIONS_DEFAULT is added in your patch but has no
> value, so you may want to provide a second version of your patch fixing
> that, including the MAINTAINER update, and posting the URL to your
> redports build in a comment, noting that its ready to commit.
>
>
Fixed.


> Running portlint (with -AC) flags is also highly recommended (add the
> output in your comment too, even if its clean)
>
> This gives committers confidence++ that your submission is ready to rock
>
>
Thanks for your useful tips, koobs@


> --
> koobs
>
>
>
> Kind regards,

--CJPM



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