Date: Sun, 27 Jan 2019 02:09:02 +0300 (MSK) From: Dmitry Morozovsky <marck@rinet.ru> To: Kirk McKusick <mckusick@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r343483 - in head/sbin: newfs tunefs Message-ID: <alpine.BSF.2.00.1901270207030.63552@woozle.rinet.ru> In-Reply-To: <201901262227.x0QMRCbv034408@repo.freebsd.org> References: <201901262227.x0QMRCbv034408@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Kirk, On Sat, 26 Jan 2019, Kirk McKusick wrote: > Author: mckusick > Date: Sat Jan 26 22:27:12 2019 > New Revision: 343483 > URL: https://svnweb.freebsd.org/changeset/base/343483 > > Log: > Update tunefs and newfs error messages for the -L (volume label) option > to note that underscores are valid. while you're here -- most of FSes also allow dash as a valid symbol in FS name. I stepped on it several times, and even did some local patch, but was too lazy to sumbit it. could you please look into it? thanks! > > PR: 235182 > Reported by: Rodney W. Grimes (rgrimes@) > Sponsored by: Netflix > > Modified: > head/sbin/newfs/newfs.c > head/sbin/tunefs/tunefs.c > > Modified: head/sbin/newfs/newfs.c > ============================================================================== > --- head/sbin/newfs/newfs.c Sat Jan 26 22:24:15 2019 (r343482) > +++ head/sbin/newfs/newfs.c Sat Jan 26 22:27:12 2019 (r343483) > @@ -155,7 +155,8 @@ main(int argc, char *argv[]) > while (isalnum(volumelabel[++i]) || > volumelabel[i] == '_'); > if (volumelabel[i] != '\0') { > - errx(1, "bad volume label. Valid characters are alphanumerics."); > + errx(1, "bad volume label. Valid characters " > + "are alphanumerics and underscores."); > } > if (strlen(volumelabel) >= MAXVOLLEN) { > errx(1, "bad volume label. Length is longer than %d.", > > Modified: head/sbin/tunefs/tunefs.c > ============================================================================== > --- head/sbin/tunefs/tunefs.c Sat Jan 26 22:24:15 2019 (r343482) > +++ head/sbin/tunefs/tunefs.c Sat Jan 26 22:27:12 2019 (r343483) > @@ -191,9 +191,8 @@ main(int argc, char *argv[]) > i = -1; > while (isalnum(Lvalue[++i]) || Lvalue[i] == '_'); > if (Lvalue[i] != '\0') { > - errx(10, > - "bad %s. Valid characters are alphanumerics.", > - name); > + errx(10, "bad %s. Valid characters are " > + "alphanumerics and underscores.", name); > } > if (strlen(Lvalue) >= MAXVOLLEN) { > errx(10, "bad %s. Length is longer than %d.", > _______________________________________________ > svn-src-all@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/svn-src-all > To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org" > -- Sincerely, D.Marck [DM5020, MCK-RIPE, DM3-RIPN] [ FreeBSD committer: marck@FreeBSD.org ] ------------------------------------------------------------------------ *** Dmitry Morozovsky --- D.Marck --- Wild Woozle --- marck@rinet.ru *** ------------------------------------------------------------------------
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.00.1901270207030.63552>