octave-dev

[OctDev] Wrong permissions in secs1d tarball

[Rafael L. <ra...@la...>]

The following files have wrong permissions in the 0.0.9 tarball of
secs1d:

    inst/secs1d_dd_gummel_map.m
    inst/secs1d_nlpoisson_newton.m
    inst/secs1d_silicon_material_properties.m

Please, fix this in the next release.

Thanks,

Rafael

Re: [OctDev] Wrong permissions in secs1d tarball

[c. <car...@gm...>]

On 2 Sep 2012, at 17:03, Rafael Laboissiere wrote:

> The following files have wrong permissions in the 0.0.9 tarball of
> secs1d:
> 
>    inst/secs1d_dd_gummel_map.m
>    inst/secs1d_nlpoisson_newton.m
>    inst/secs1d_silicon_material_properties.m
> 
> Please, fix this in the next release.
> 
> Thanks,
> 
> Rafael

thanks, I fixed the permissions in my local copy.
As that is where I make the release from I think the next release will have correct permissions.
c.

Re: [OctDev] Wrong permissions in secs1d tarball

[Thomas W. <tw...@de...>]

On Sun, Sep 02, 2012 at 08:08:16PM +0200, c. wrote:
> 
> On 2 Sep 2012, at 17:03, Rafael Laboissiere wrote:
> 
> > The following files have wrong permissions in the 0.0.9 tarball of
> > secs1d:
> > 
> >    inst/secs1d_dd_gummel_map.m
> >    inst/secs1d_nlpoisson_newton.m
> >    inst/secs1d_silicon_material_properties.m
> > 
> > Please, fix this in the next release.
> > 
> > Thanks,
> > 
> > Rafael
> 
> thanks, I fixed the permissions in my local copy.
> As that is where I make the release from I think the next release will have correct permissions.

I think the permissions should be fixed in the SVN repository, too. I
can do this, but I'm hesitating just in case I'm missing something here.

	Thomas

Re: [OctDev] Wrong permissions in secs1d tarball

[c. <car...@gm...>]

On 2 Sep 2012, at 20:38, Thomas Weber wrote:

> On Sun, Sep 02, 2012 at 08:08:16PM +0200, c. wrote:
>> 
>> On 2 Sep 2012, at 17:03, Rafael Laboissiere wrote:
>> 
>>> The following files have wrong permissions in the 0.0.9 tarball of
>>> secs1d:
>>> 
>>>   inst/secs1d_dd_gummel_map.m
>>>   inst/secs1d_nlpoisson_newton.m
>>>   inst/secs1d_silicon_material_properties.m
>>> 
>>> Please, fix this in the next release.
>>> 
>>> Thanks,
>>> 
>>> Rafael
>> 
>> thanks, I fixed the permissions in my local copy.
>> As that is where I make the release from I think the next release will have correct permissions.
> 
> I think the permissions should be fixed in the SVN repository, too. I
> can do this, but I'm hesitating just in case I'm missing something here.
I changed the permissions locally and tried to commit the changes but it seems nothing happened.
I'm not sure if/how file permissions can be stored in subversion, the executable property can be set 
with "propset" but I don't know how to change the other permissions.
If you do please go ahead and change them.

Thanks,
c. 

> 	Thomas

Re: [OctDev] Wrong permissions in secs1d tarball

[Thomas W. <tw...@de...>]

On Sun, Sep 02, 2012 at 09:39:03PM +0200, c. wrote:
> I changed the permissions locally and tried to commit the changes but it seems nothing happened.
> I'm not sure if/how file permissions can be stored in subversion, the executable property can be set 
> with "propset" but I don't know how to change the other permissions.
> If you do please go ahead and change them.

Okay, it seems that I indeed don't understand something here. I thought
that Rafael just meant the executable bit. Rafael, what are the
permissions you are talking about? Apart from the executable bit, the
permissions in the tarball are 644 after unpacking, which seems
appropriate.

	Thomas

Re: [OctDev] Wrong permissions in secs1d tarball

[c. <car...@gm...>]

On 3 Sep 2012, at 20:04, Thomas Weber wrote:

> Okay, it seems that I indeed don't understand something here. I thought
> that Rafael just meant the executable bit. Rafael, what are the
> permissions you are talking about? Apart from the executable bit, the
> permissions in the tarball are 644 after unpacking, which seems
> appropriate.

In my local copy some of the files listed by Raphael had mode 640, 
is this not the case in the tarball?

> 	Thomas
c.

Re: [OctDev] Wrong permissions in secs1d tarball

[Rafael L. <ra...@la...>]

* Thomas Weber <tw...@de...> [2012-09-03 20:04]:

> On Sun, Sep 02, 2012 at 09:39:03PM +0200, c. wrote:
> > I changed the permissions locally and tried to commit the changes but it seems nothing happened.
> > I'm not sure if/how file permissions can be stored in subversion, the executable property can be set 
> > with "propset" but I don't know how to change the other permissions.
> > If you do please go ahead and change them.
> 
> Okay, it seems that I indeed don't understand something here. I thought
> that Rafael just meant the executable bit. Rafael, what are the
> permissions you are talking about? Apart from the executable bit, the
> permissions in the tarball are 644 after unpacking, which seems
> appropriate.

I see permissions 755 for some .m files in the tarball:

$ tar tfvz secs1d-0.0.9.tar.gz | grep inst 
drwxr-xr-x carlo/staff       0 2012-03-30 00:56 secs1d-0.0.9/inst/
-rwxr-xr-x carlo/staff   10663 2012-03-30 00:54 secs1d-0.0.9/inst/secs1d_dd_gummel_map.m
-rw-r--r-- carlo/staff    8319 2012-03-30 00:54 secs1d-0.0.9/inst/secs1d_dd_newton.m
-rwxr-xr-x carlo/staff    5653 2012-03-30 00:54 secs1d-0.0.9/inst/secs1d_nlpoisson_newton.m
-rw-r--r-- carlo/staff    1256 2012-03-26 00:44 secs1d-0.0.9/inst/secs1d_physical_constants.m
-rwxr-xr-x carlo/staff    3750 2012-03-26 00:44 secs1d-0.0.9/inst/secs1d_silicon_material_properties.m

Rafael

Re: [OctDev] Wrong permissions in secs1d tarball

[c. <car...@gm...>]

On 3 Sep 2012, at 22:12, Rafael Laboissiere wrote:

> * Thomas Weber <tw...@de...> [2012-09-03 20:04]:
> 
>> On Sun, Sep 02, 2012 at 09:39:03PM +0200, c. wrote:
>>> I changed the permissions locally and tried to commit the changes but it seems nothing happened.
>>> I'm not sure if/how file permissions can be stored in subversion, the executable property can be set 
>>> with "propset" but I don't know how to change the other permissions.
>>> If you do please go ahead and change them.
>> 
>> Okay, it seems that I indeed don't understand something here. I thought
>> that Rafael just meant the executable bit. Rafael, what are the
>> permissions you are talking about? Apart from the executable bit, the
>> permissions in the tarball are 644 after unpacking, which seems
>> appropriate.
> 
> I see permissions 755 for some .m files in the tarball:
> 
> $ tar tfvz secs1d-0.0.9.tar.gz | grep inst 
> drwxr-xr-x carlo/staff       0 2012-03-30 00:56 secs1d-0.0.9/inst/
> -rwxr-xr-x carlo/staff   10663 2012-03-30 00:54 secs1d-0.0.9/inst/secs1d_dd_gummel_map.m
> -rw-r--r-- carlo/staff    8319 2012-03-30 00:54 secs1d-0.0.9/inst/secs1d_dd_newton.m
> -rwxr-xr-x carlo/staff    5653 2012-03-30 00:54 secs1d-0.0.9/inst/secs1d_nlpoisson_newton.m
> -rw-r--r-- carlo/staff    1256 2012-03-26 00:44 secs1d-0.0.9/inst/secs1d_physical_constants.m
> -rwxr-xr-x carlo/staff    3750 2012-03-26 00:44 secs1d-0.0.9/inst/secs1d_silicon_material_properties.m
> 
> Rafael

I removed the property svn:executable (which was indeed set) from the files

inst/secs1d_dd_gummel_map.m
inst/secs1d_nlpoisson_newton.m
inst/secs1d_silicon_material_properties.m

I hope this fixes the problem.
Thanks!
c.