Date: Fri, 07 Aug 2020 09:45:05 -0600 From: Ian Lepore <ian@freebsd.org> To: Emmanuel Vadot <manu@bidouilliste.com>, rgrimes@freebsd.org Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r364010 - head/sbin/iscontrol Message-ID: <5456249cc4128924a57729905b561ad39bc8663e.camel@freebsd.org> In-Reply-To: <20200807173214.0f831da5ab649eec811b2466@bidouilliste.com> References: <202008071219.077CJLbH085411@repo.freebsd.org> <202008071526.077FQ0uQ017440@gndrsh.dnsmgr.net> <20200807173214.0f831da5ab649eec811b2466@bidouilliste.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 2020-08-07 at 17:32 +0200, Emmanuel Vadot wrote: > On Fri, 7 Aug 2020 08:26:00 -0700 (PDT) > "Rodney W. Grimes" <freebsd@gndrsh.dnsmgr.net> wrote: > > > > Author: manu > > > Date: Fri Aug 7 12:19:21 2020 > > > New Revision: 364010 > > > URL: https://svnweb.freebsd.org/changeset/base/364010 > > > > > > Log: > > > pkgbase: We can't easily have a package with either a - or a _ > > > > Wow, hopefully this is short term. > > Not really. > > > I would think a package name can be any valid file name, and to > > remove - and _ from that set is going > > to cause lots of POLA. > > The package file itself contain '-', they are named > FreeBSD-<name>-{dbg,lib32,dev,}-<version>. it's just that we used - > and > _ when building them to get some variable which contain the > description, nothing that can't be patched by this is clearly not my > priority. > This should be easy to fix. Use sed instead of tr, and translate - to __ to remove dashes, and then translate __ back to - to restore them, and you've properly preserved single _ usage in names. -- Ian > > > > > > > > Rename iscsi_legacy to iscsilegacy, having - or _ in a package > > > name cause > > > problems when we process them and generate the ucl. > > > > > > Modified: > > > head/sbin/iscontrol/Makefile > > > > > > Modified: head/sbin/iscontrol/Makefile > > > ================================================================= > > > ============= > > > --- head/sbin/iscontrol/Makefile Fri Aug 7 10:20:39 2020 ( > > > r364009) > > > +++ head/sbin/iscontrol/Makefile Fri Aug 7 12:19:21 2020 ( > > > r364010) > > > @@ -1,6 +1,6 @@ > > > # $FreeBSD$ > > > > > > -PACKAGE=iscsi_legacy > > > +PACKAGE=iscsilegacy > > > SRCS= iscontrol.c pdu.c fsm.c config.c login.c auth_subr.c > > > misc.c > > > PROG= iscontrol > > > LIBADD= cam md > > > > > > > -- > > Rod Grimes > > rgrimes@freebsd.org > >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5456249cc4128924a57729905b561ad39bc8663e.camel>