Saturday, June 11, 2022

Re: [new] www/freshrss-1.19.2 rss/atom feed aggregator

aisha <openbsd.ports@aisha.cc> wrote:
> On 5/21/2022 8:00 AM, aisha wrote:
> > Fixed!
> > Updated tarball attached.
> >
> > Thanks,
> > Aisha
>
> ping
>
> aisha

two small nitpicks and one personal preference (that's opinable):

- the license is AGPL-3.0. (for stuff like this the difference between
GPL and affero gpl is important.)

- I'd add PKG_ARCH = *

- I don't particularly like "FreshRSS" as package name, what about
"freshrss" (lowercase)? Setting FULLPKGNAME=${PKGNAME:L} should do it
in this case.

otherwise seems OK for me, I still haven't tested at runtime but will
probably in the next weeks :)

(no need to send another tarball, i trust whoever imports it to do this
two small fixes.)

Thanks,

Omar Polo

No comments:

Post a Comment