Date: Mon, 27 Feb 2017 18:20:03 +0100
From: Natanael Copa <ncopa@alpinelinux.org>
To: Valery Kartel <valery.kartel@gmail.com>
Cc: =?ISO-8859-1?B?U/ZyZW4=?= Tempel <soeren@soeren-tempel.net>,
 alpine-aports@lists.alpinelinux.org
Subject: Re: [alpine-aports] [PATCH] main/perl: restore the current
 compatibility
Message-ID: <20170227182003.53b24e1d@ncopa-desktop.copa.dup.pw>
In-Reply-To: <20170213125844.GA19524@francium>
References: <20170213080258.28519-1-valery.kartel@gmail.com>
	<20170213125844.GA19524@francium>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On Mon, 13 Feb 2017 13:58:44 +0100
S=F6ren Tempel <soeren@soeren-tempel.net> wrote:

> Thanks for providing a patch that quickly.
>=20
> On 13.02.17, Valery Kartel wrote:
> >       local file
> > -     for file in c2ph corelist enc2xs instmodsh h2ph h2xs libnetcfg
> > perldoc \
> > -             perlivp perlthanks pl2pm pod2html pod2man pod2text
> > pod2usage \
> > -             podchecker podselect xsubpp $pkgname$pkgver
> > +     for file in c2ph instmodsh h2ph h2xs libnetcfg perlivp
> > perlthanks \
> > +             pl2pm pod2html pod2usage podchecker podselect xsubpp
> > $pkgname$pkgver
> >       do
> >               mv "$pkgdir"/usr/bin/$file "$subpkgdir"/usr/bin/ ||
> >               return 1
> >       done =20
>=20
> What about the other pod* scripts? pod2html, pod2usage, ...
> Shouldn't they be moved back to the origin package as well?

I just got bitten by this for xen.

Where do we want to have pod* scripts?

Should we fix the APKBUILDs to depend on perl-dev to be able to
generate manual pages with pod2man?

Should we have a separate pod* package?

Should we ship those tools with 'perl' like we did earlier?

The commit log does not explain *why* those changes was made in the
first place so I am afraid of just reverting.

Valery, what problem did you try to solve by moving pod* scripts to
-dev?

It would also be great if you could also explain *why* we need
to restore the current compatibility in the commit message for the
"main/perl: restore the current compatibility" patch.

Thanks!

-nc


---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---