This is the mail archive of the
cygwin
mailing list for the Cygwin project.
Re: ATTENTION JARI AALTO Re: quilt: permission denied problems with helper scripts
[I already had Jari on bcc, given the track record, but I
managed not to scream :-) He's on bcc this time too]
On 2012-09-14 16:16, Christopher Faylor wrote:
> On Fri, Sep 14, 2012 at 01:05:08PM +0200, Peter Rosin wrote:
>> Hi!
>>
>> When working on some newlib patches, ran into this:
>>
>> peda@pulkan ~/newlib/src/newlib
>> $ quilt refresh
>> /usr/share/quilt/refresh: line 253: /usr/share/quilt/scripts/remove-trailing-ws: Permission denied
>> Patch patches/wordexp-avoid-strdup.patch is unchanged
>>
>> peda@pulkan ~/newlib/src/newlib
>> $ ls -lG /usr/share/quilt/scripts
>> total 68
>> -rwxr-xr-x 1 peda 6219 May 24 2011 backup-files
>> -rw-r--r-- 1 peda 13114 May 24 2011 dependency-graph
>> -rw-r-xr-x 1 peda 5910 May 24 2011 edmail
>> -rwxr-xr-x 1 peda 4642 May 24 2011 inspect
>> -rw-r--r-- 1 peda 3397 May 24 2011 parse-patch
>> -rw-r--r-- 1 peda 18389 May 24 2011 patchfns
>> -rw-r--r-- 1 peda 3491 May 24 2011 remove-trailing-ws
>>
>> All those files are scripts with shebangs, and I assume they
>> should all be chmod +x, and indeed that fixes the "quilt refresh"
>> problem (and a similar problem in "quilt mail" when it tries to
>> execute edmail).
>>
>> Please fix the packaging bug.
Oh, forgot to mention, "quilt mail" expects sensible-editor to
be a valid command (there are other places too, "quilt edit" and
"quilt header", if I grep for a bit), which seems to be some kind
of approach to do what $EDITOR normally does but perhaps using
alternatives instead?
But, Cygwin doesn't provide sensible-editor anywhere, so I had
to manually create a symlink for that to work. Here's my
workaround in case any quilt user needs it (which seems unlikely):
ln -s /usr/bin/vim /usr/local/bin/sensible-editor
A peek inside the quilt package reveals that sensible-editor is
a debianism and that upstream indeed uses $EDITOR. Jari, please
scratch the use-sensible-editor patch, or invent something that
provides sensible-editor and add that to Requires:
Also, upstream is at 0.60, so an update from 0.48 is overdue.
Cheers,
Peter
--
Problem reports: http://cygwin.com/problems.html
FAQ: http://cygwin.com/faq/
Documentation: http://cygwin.com/docs.html
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple