On Sat, Aug 20, 2016 at 1:46 AM tim Rowledge <tim at rowledge.org> wrote:

>
> > On 19-08-2016, at 4:32 PM, Levente Uzonyi <leves at caesar.elte.hu> wrote:
> >
> > On Fri, 19 Aug 2016, Fabio Niephaus wrote:
> >
> >> On Fri, Aug 19, 2016 at 11:31 PM tim Rowledge <tim at rowledge.org> wrote:
> >>
> >>      > On 19-08-2016, at 12:41 PM, Fabio Niephaus <lists at fniephaus.com>
> wrote:
> >>      >
> >>      > Thanks for the bug reports. I've fixed the regex bug and found a
> way to ensure "find" ignores dot files.
> >>
> >>      Excellent.
> >>
> >>      > But why do you think ensure_conf_file needs to depend upon the
> kernel level?
> >>
> >>      Because the issue was fixed in the kernel some time ago.
> Unfortunately I couldn’t tell you exactly when, but the Pi stopped needing
> the file addition around 18 months ago. That’s around the time
> >>      the kernel was bumped up to v 4 I think. Whether it is any sort of
> problem to have such a file when it isn’t required… I don’t know. It’s
> certainly less alarming for any user if the script doesn’t
> >>      demand their superuser password!
> >> Can someone shed some light on this or do some more digging? When
> exactly do we need a squeak.conf? Any other ideas how to deal with this?
> >
> > You always need it when you use an ht VM. Without the file the VM will
> not start (unless you run it as root, but that's something you wouldn't do,
> would you?).
> > All VMs built by the CI are ht VMs, so the file is a must.
>
>
> This is not strictly true; recent kernels simply don’t cause the problem.
> My Pi just doesn’t need the file and hasn’t for 18+ months. I’ve checked
> for it pre-existing and it doesn’t. I’ve been delivering the PI system with
> an ht based vm for a long time now and millions of people use it daily.
>

May I suggest we simply skip the ensure_conf_file step, if the launch
script detects to run on a Pi for this release? Shall we print instructions
for the common-session setup if necessary instead? We can adjust this
behavior in the future, of course. But I'm not sure we can work out a
proper solution in the next couple of days.

Fabio


>
>
> > The file has no effect on kernels before 2.6.X (basically 10+ years
> old), so checking the version is unnecessary (the VM would probably not
> start anyway due to C library differences).
> >
> > Levente
> >
> >> Fabio
> >>
> >>
> >>      tim
> >>      --
> >>      tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim
> >>      Oxymorons: Good grief
> >
>
>
> tim
> --
> tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim
> To define recursion, we must first define recursion.
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20160820/b487d58a/attachment.htm