On Mon, Sep 10, 2018 at 08:17:09AM +0200, Theo Buehler wrote:
> On Sat, Sep 08, 2018 at 04:02:10AM -0600, Anthony J. Bentley wrote:
> > Hi,
> > 
> > The Nu Html Checker (v.Nu) is the backend of checker.html5.org,
> > html5.validator.nu, and validator.w3.org/nu. It can be used to batch-check
> > HTML 5 documents from the command line and from other scripts/apps, or
> > deployed as a self-contained web service.
> > 
> > 
> > Two simple ways to test: validate files with "vnu *.html" or "vnu foo/";
> > or rcctl start vnu and visit localhost:8888 in a browser to use the web
> > interface.
> > 
> > ok?
> 
> This line in pkg/PLIST 
> 
> @newuser _vnu:818:818:daemon:Nu Html Checker:${VARBASE}/empty:/sbin/nologin
> 
> ended up verbatim in my master.passwd file (without VARBASE expanded),
> thus 'rcctl -df start vnu' failde with
> 
> ...
> doing rc_check
> su: unveil: No such file or directory
> doing _rc_rm_runfile
> (failed)
> 
> I think ${VARBASE}/empty should be replaced with a simple '/var/empty'
> in that line.
> 
> With that fixed,
> 
> ok tb.

Agreed. /var/empty is part of the base system. So it should always be
/var/empty whatever the value of VARBASE is.

I really think we are going too far with all these variables...

-- 
Antoine

Reply via email to