On Mon, 2015-10-12 at 15:21 +0200, Samuel Thibault wrote:
> I believe they will. But perhaps in your tests they didn't (but did
> your tests really try that?), in which case it'd mean there is a bug.
> But for now nothing said there was a bug there.
FYI: gcc-snapshot (20150817, not 20150913)
Svante Signell, le Tue 13 Oct 2015 13:42:22 +0200, a écrit :
> On Mon, 2015-10-12 at 15:21 +0200, Samuel Thibault wrote:
>
> > I believe they will. But perhaps in your tests they didn't (but did
> > your tests really try that?), in which case it'd mean there is a bug.
> > But for now nothing
On Tue, 2015-10-13 at 13:49 +0200, Samuel Thibault wrote:
> Svante Signell, le Tue 13 Oct 2015 13:42:22 +0200, a écrit :
> > On Mon, 2015-10-12 at 15:21 +0200, Samuel Thibault wrote:
> > FYI: gcc-snapshot (20150817, not 20150913) and frama-c (20150201+sodium
> > +dfsg-2) with downgraded
Svante Signell, le Mon 12 Oct 2015 15:16:19 +0200, a écrit :
> On Mon, 2015-10-12 at 15:12 +0200, Samuel Thibault wrote:
> > Svante Signell, le Mon 12 Oct 2015 15:04:40 +0200, a écrit :
> > > OK I buy your explanation; though there seems to be remaining bugs.
> >
> > Which bugs?
>
> You wrote it
On Mon, 2015-10-12 at 14:49 +0200, Samuel Thibault wrote:
> Svante Signell, le Mon 12 Oct 2015 14:45:09 +0200, a écrit :
> > On Mon, 2015-10-12 at 02:08 +0200, Samuel Thibault wrote:
> > > The commit I have just pushed, which does make sense, does fix gpsd at
> > > least.
> Please erase your
On Mon, 2015-10-12 at 02:08 +0200, Samuel Thibault wrote:
> The commit I have just pushed, which does make sense, does fix gpsd at
> least.
Building gpsd (3.15-1) with your patch applied:
cp /home/srs/DEBs/gpsd/gpsd-3.15/gpsd.rules
/home/srs/DEBs/gpsd/gpsd-3.15/debian/gpsd.udev
cp: cannot create
On Mon, 2015-10-12 at 15:12 +0200, Samuel Thibault wrote:
> Svante Signell, le Mon 12 Oct 2015 15:04:40 +0200, a écrit :
> > OK I buy your explanation; though there seems to be remaining bugs.
>
> Which bugs?
You wrote it not me:
> Subsequent writes to the file will then fail (and if they don't,
Svante Signell, le Mon 12 Oct 2015 15:04:40 +0200, a écrit :
> OK I buy your explanation; though there seems to be remaining bugs.
Which bugs?
Samuel
Svante Signell, le Mon 12 Oct 2015 14:45:09 +0200, a écrit :
> On Mon, 2015-10-12 at 02:08 +0200, Samuel Thibault wrote:
> > The commit I have just pushed, which does make sense, does fix gpsd at
> > least.
>
> Building gpsd (3.15-1) with your patch applied:
> cp
Samuel Thibault, le Mon 12 Oct 2015 14:49:43 +0200, a écrit :
> Your proposed change does not meet what I said.
> Notably, in the nn->openmodes = O_READ and newmodes = O_WRITE case,
> it'll be wrong: nn->openmodes & newmodes will be zero, and thus the
> 'file' port will be used as such, but it
Svante Signell, le Sun 11 Oct 2015 23:02:16 +0200, a écrit :
> > We also see from the printout that
> > nn->openmodes = 2 = O_WRITE and
> > newmodes = 1 = O_READ i.e. no intersecting sets.
> >
>
> The above condition is really happening when building (patched to
> build, not related to fakeroot)
On Thu, 2015-05-21 at 10:56 +0200, Svante Signell wrote:
> On Fri, 2015-05-15 at 20:31 +0200, Samuel Thibault wrote:
> > Svante Signell, le Tue 12 May 2015 09:15:46 +0200, a écrit :
>
> > > - if (file != MACH_PORT_NULL && (nn->openmodes & ~newmodes))
> > > + if (file != MACH_PORT_NULL
The commit I have just pushed, which does make sense, does fix gpsd at
least.
Samuel
On Tue, 2015-06-09 at 17:15 +0200, Svante Signell wrote:
I've built pycorrfit and hurd so far, no issues yet. Will build glibc,
gnat-4.9 and gcc-5 to be sure.
glibc and gnat-4.9 built fine. gcc-5 will take several hours.
On Wed, 2015-06-10 at 10:05 +0200, Svante Signell wrote:
On Tue, 2015-06-09 at 17:15 +0200, Svante Signell wrote:
I've built pycorrfit and hurd so far, no issues yet. Will build glibc,
gnat-4.9 and gcc-5 to be sure.
glibc and gnat-4.9 built fine. gcc-5 will take several hours.
And
On Tue, 2015-06-09 at 11:11 +0200, Samuel Thibault wrote:
So the package is actually doing something stupid (yes, that's what you
should have written in your mail to explain what is happening, actually
:) ). But it does work as root as specified by POSIX, so we have to
support it.
Yes I
On Tue, 2015-06-09 at 10:55 +0200, Svante Signell wrote:
On Tue, 2015-06-09 at 10:43 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 09 Jun 2015 10:13:46 +0200, a écrit :
Done so now. Attached are two versions of a patch to fakeroot.c. In
netfs_get_dirents(): call
Svante Signell, le Tue 09 Jun 2015 10:55:35 +0200, a écrit :
On Tue, 2015-06-09 at 10:43 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 09 Jun 2015 10:13:46 +0200, a écrit :
Done so now. Attached are two versions of a patch to fakeroot.c. In
netfs_get_dirents(): call
On Fri, 2015-06-05 at 11:18 +0530, Samuel Thibault wrote:
Hello,
It seems there is still a corner case which doesn't work: the pycorrfit
package fails to build with:
chmod -R 644
debian/pycorrfit/usr/share/doc/pycorrfit/examples/external_model_functions/
chmod: cannot access
Svante Signell, le Tue 09 Jun 2015 10:13:46 +0200, a écrit :
Done so now. Attached are two versions of a patch to fakeroot.c. In
netfs_get_dirents(): call netfs_attempt_chmod() before dir_readdir() to
make sure that directories are accessible (executable) before changing
the underlying files.
On Tue, 2015-06-09 at 10:43 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 09 Jun 2015 10:13:46 +0200, a écrit :
Done so now. Attached are two versions of a patch to fakeroot.c. In
netfs_get_dirents(): call netfs_attempt_chmod() before dir_readdir() to
make sure that directories are
Svante Signell, le Tue 09 Jun 2015 11:41:01 +0200, a écrit :
On Tue, 2015-06-09 at 11:11 +0200, Samuel Thibault wrote:
So the package is actually doing something stupid (yes, that's what you
should have written in your mail to explain what is happening, actually
:) ). But it does work as
Hello,
It seems there is still a corner case which doesn't work: the pycorrfit
package fails to build with:
chmod -R 644
debian/pycorrfit/usr/share/doc/pycorrfit/examples/external_model_functions/
chmod: cannot access
gnat-5 built fine indeed.
Congrats again for having tracked this bug!
I have switched the buildds to using fakeroot-hurd by default, and will
probably bump the alternatives priority soon too.
Samuel
Quoting Samuel Thibault (2015-05-23 23:53:07)
I have switched the buildds to using fakeroot-hurd by default, and will
probably bump the alternatives priority soon too.
Awesome :)
Justus
Thanks again for the explanations, that's exactly what I needed to
manage to have a look at your proposed changes quickly :)
Svante Signell, le Thu 21 May 2015 10:56:24 +0200, a écrit :
- there are two cosmetic changes (I know they should not be there, but
for code unification/readability)
-
Samuel Thibault, le Sat 23 May 2015 02:04:24 +0530, a écrit :
Could you try the attached patch? It makes your testcase work fine, but
perhaps it has other unexpected consequences.
Thinking again about it, I believe it will be really fine, so commited
it.
Samuel
On Fri, 2015-05-15 at 20:31 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 12 May 2015 09:15:46 +0200, a écrit :
- if (file != MACH_PORT_NULL (nn-openmodes ~newmodes))
+ if (file != MACH_PORT_NULL (nn-openmodes newmodes )) /* works
*/
This change needs to be
Svante Signell, le Tue 12 May 2015 09:15:46 +0200, a écrit :
+ /* XXX: Limit openmodes */
+ if (openmodes O_ALLOWED)
+openmodes = O_ALLOWED; /* works */
More precisely, which openmode posed problem here? Perhaps this is not
the best place to exclude them. At the very least we need to
Svante Signell, le Tue 12 May 2015 09:15:46 +0200, a écrit :
The idea is to limit the openmodes according to the values defined in
struct netnode and to change the test for overlapping sets.
In which case is this needed? What happens in that case and how the
patch fixes this?
A patch should be
On Tue, 2015-05-12 at 09:42 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 12 May 2015 09:15:46 +0200, a écrit :
The idea is to limit the openmodes according to the values defined in
struct netnode and to change the test for overlapping sets.
In which case is this needed? What
On Tue, 2015-05-12 at 11:22 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 12 May 2015 10:00:26 +0200, a écrit :
On Tue, 2015-05-12 at 09:42 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 12 May 2015 09:15:46 +0200, a écrit :
The idea is to limit the openmodes according to
Svante Signell, le Tue 12 May 2015 10:00:26 +0200, a écrit :
On Tue, 2015-05-12 at 09:42 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 12 May 2015 09:15:46 +0200, a écrit :
The idea is to limit the openmodes according to the values defined in
struct netnode and to change the test
Svante Signell, le Tue 12 May 2015 11:39:19 +0200, a écrit :
On Tue, 2015-05-12 at 11:22 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 12 May 2015 10:00:26 +0200, a écrit :
On Tue, 2015-05-12 at 09:42 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 12 May 2015 09:15:46 +0200,
Svante Signell, le Tue 12 May 2015 10:00:26 +0200, a écrit :
On Tue, 2015-05-12 at 09:42 +0200, Samuel Thibault wrote:
Svante Signell, le Tue 12 May 2015 09:15:46 +0200, a écrit :
The idea is to limit the openmodes according to the values defined in
struct netnode and to change the test
35 matches
Mail list logo