This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]