Date: Sat, 06 Oct 2012 22:24:58 -0700 From: Doug Barton <dougb@FreeBSD.org> To: Jason Helfman <jgh@FreeBSD.org> Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, nemysis@gmx.ch, ports-committers@freebsd.org Subject: Re: svn commit: r305338 - in head/games/pycadia: . files Message-ID: <5071122A.1050209@FreeBSD.org> In-Reply-To: <CAMuy=%2Bg9OiTEFrKy1KB6Aju-rtKyG4dvNBt7zi4zDgGpQRcvxg@mail.gmail.com> References: <201210060537.q965bUpk005984@svn.freebsd.org> <5070A09B.1090404@FreeBSD.org> <CAMuy=%2BiTW7AgjW1QGjS9n9E5hXfwLwq=kKxmBSia2nDKs_-9-g@mail.gmail.com> <5070B0F0.6080404@FreeBSD.org> <CAMuy=%2Bi15nHdSsyBKo7C4h4jBcGX=ji7ixajhw5W9nXsVHwH5A@mail.gmail.com> <5070B355.1000802@FreeBSD.org> <CAMuy=%2Bg9OiTEFrKy1KB6Aju-rtKyG4dvNBt7zi4zDgGpQRcvxg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 10/06/2012 17:19, Jason Helfman wrote: > On Sat, Oct 6, 2012 at 3:40 PM, Doug Barton <dougb@freebsd.org > <mailto:dougb@freebsd.org>> wrote: > > On 10/06/2012 15:34, Jason Helfman wrote: > > On Sat, Oct 6, 2012 at 3:30 PM, Doug Barton <dougb@freebsd.org > <mailto:dougb@freebsd.org> > > <mailto:dougb@freebsd.org <mailto:dougb@freebsd.org>>> wrote: > > > > On 10/06/2012 15:25, Jason Helfman wrote: > > > On Sat, Oct 6, 2012 at 2:20 PM, Doug Barton > <dougb@freebsd.org <mailto:dougb@freebsd.org> > > <mailto:dougb@freebsd.org <mailto:dougb@freebsd.org>> > > > <mailto:dougb@freebsd.org <mailto:dougb@freebsd.org> > <mailto:dougb@freebsd.org <mailto:dougb@freebsd.org>>>> wrote: > > > > > > On 10/05/2012 22:37, Jason Helfman wrote: > > > > Added: head/games/pycadia/files/pycadia.sh.in > <http://pycadia.sh.in>; > > <http://pycadia.sh.in>; <http://pycadia.sh.in>; > > > > > > > > > > ============================================================================== > > > > --- /dev/null 00:00:00 1970 (empty, because file is > newly > > added) > > > > +++ head/games/pycadia/files/pycadia.sh.in > <http://pycadia.sh.in>; > > <http://pycadia.sh.in>; <http://pycadia.sh.in>; > > > Sat Oct 6 05:37:29 2012 (r305338) > > > > @@ -0,0 +1,6 @@ > > > > +#!/bin/sh > > > > +# > > > > +# $FreeBSD$ > > > > + > > > > +cd "%%DATADIR%%" > > > > +./pycadia.py > > > > > > Unless there is some reason to cd into that directory, you'd > > be better > > > off with just: %%DATADIR%%/pycadia.py > > > > > > And in any case, shouldn't you be using exec here? > > > > > > > > > I can check with the maintainer, but it does need to be run > from the > > > DATADIR specifically. > > > > Ok, that's not a problem. A comment to that effect would > probably be > > useful for future reference. > > > > > > > What would exec provide that isn't currently fulfilled? > > > > Eliminating the shell. > > > > Fair enough. I can discuss with maintainer, as he has many ports > that do > > this. I would suggest using %%PYTHON_CMD%% in favor of exec. Do > you see > > an issue with that? > > Yes, unless %%PYTHON_CMD%% has some magic to eliminate the shell that > I'm not aware of. :) > > Try the existing command with and without exec and then look at your ps > output to see my point. > > Doug > > > Currently I'm not aware of a method to eliminate the shell within a > shell script. None of us knows everything (including me). :) > Had I realized this was such an issue I would've sent mail > requesting a review. It's not that big a deal. > %%PYTHON_CMD%% become /usr/local/bin/pythonXX Still missing the point. :) I sent you and the maintainer a private message spelling it all out in nauseating detail. Doug -- I am only one, but I am one. I cannot do everything, but I can do something. And I will not let what I cannot do interfere with what I can do. -- Edward Everett Hale, (1822 - 1909)
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5071122A.1050209>