On Fri, 23 Aug 2024 22:00:22 +0200
Omar Polo <o...@omarpolo.com> wrote:
> On 2024/08/23 21:55:41 +0200, Kirill A. Korinsky <kir...@korins.ky>
> wrote:
> > On Fri, 23 Aug 2024 21:31:32 +0200,
> > Omar Polo <o...@omarpolo.com> wrote:
> > >
> > > I'm sorry I don't know enough about perl packaging to answer.
> > > However, if this is the perl module default, I guess it makes
> > > sense?
> >
> > I didn't mean to offend in any way, and I've just checked a random
> > perl port which was mail/p5-Mail-IMAPClient and it contains:
> >
> > BUILD_DEPENDS= ${RUN_DEPENDS}
> >
> > So, maybe it should be fixed as well.
> >
> > I really don't know and just shared that I had noticed when I tried
> > it.
> >
> > My comment shouldn't be taken as something that prevents an update
> > in any way.
>
> Don't worry, it's great to take the chance to improve a port while
> upgraditing it!
>
> I'm not feeling strong about this, these run depends are not going to
> do anything at build time, but I can see how it may help in not
> missing new deps on future updates.
Port works fine for me, though I would prefer not setting BUILD_DEPENDS
to RUN_DEPENDS unless this is common practice. It makes the port more
confusing to readers, just to remove a benign warning from the build
script.
