Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 22 Sep 2013 11:55:26 -0400
From:      Eitan Adler <lists@eitanadler.com>
To:        Matthew Windsor <mbw500@york.ac.uk>
Cc:        ports@freebsd.org
Subject:   Re: [CFT] pkgpackagekit (GSoC: PackageKit backend for pkgng)
Message-ID:  <CAF6rxg=XJ0a4O3E3=xJ6jHv1Nfm8aqBfyxpcQoxKj4G4kdk3uA@mail.gmail.com>
In-Reply-To: <CAFxS2CgLNiX0op3_ZYVeZqpN=VvoXvHKDYeNqdEjCjPnmWC%2BLg@mail.gmail.com>
References:  <CAFxS2CgLNiX0op3_ZYVeZqpN=VvoXvHKDYeNqdEjCjPnmWC%2BLg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Sep 22, 2013 at 5:20 AM, Matthew Windsor <mbw500@york.ac.uk> wrote:
> OK, I've updated the port (not the distfiles) based on Ruslan's
> feedback; the master sites have also been changed thanks to my
> mentor's uploading of the distfiles to FreeBSD mirrors (thanks!)
>
> Not sure if this warrants a PORTVERSION bump (add a _1 to the
> version?)

This is not required.  (a) it only gets bumped when the port needs to be rebuilt
(b) typically, testing ports don't bother at all.

> as there are no changes to the distfiles and the only
> change to the way the port builds is that I added a USE_LDCONFIG to
> keep portlint happy (I'm not sure if I can disable this, since
> PackageKit is the only thing that uses the .so and it looks in a
> specific directory for it it doesn't seem necessary).

USE_LDCONFIG is not required and the warning from portlint is bogus in
this case.
I don't know of a way to silence it. :(

> The new port is now in the SVN repository; attached is a diff of it
> against an *empty directory*.  GMail seems to like collapsing tabs and
> wrapping lines (either on the email reading side or on the email
> sending side, not sure) so I'll abandon trying to inline them.  Future
> changes will probably be distributed as proper patches.

Thanks!



-- 
Eitan Adler



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