After much git heroism (new desktop) a new version of translatesieve is
available here:
https://github.com/cyrusimap/cyrus-imapd/pull/1792
Comments please, or merge.
-nic
On 01/13/2017 12:39 PM, Nic Bernstein via Cyrus-devel wrote:
On 01/13/2017 07:00 AM, Nic Bernstein via Cyrus-devel
On 01/13/2017 07:00 AM, Nic Bernstein via Cyrus-devel wrote:
I can (and, honestly, already have) fix translatesieve, and a similar
fix can be added to upgradesieve or any other of these scripts. I
think rehash should be left alone, as anyone who's already used
hashing has what they have, which
Bron,
I'll agree that there are good reasons to keep this part (i.e. systems
sans shebang):
exec perl -x -S $0 ${1+"$@"} # -*-perl-*-
Or even to keep the second chunk, which searches for a Perl 5 interpreter:
if ($] !~ /^5\..*/) {
# uh-oh. this isn't perl 5.
foreach (split(/
I can (and, honestly, already have) fix translatesieve, and a similar
fix can be added to upgradesieve or any other of these scripts. I think
rehash should be left alone, as anyone who's already used hashing has
what they have, which means they may already have upper-case
directories. Let's n
It's there because lots of people on non-Linux systems have their perl
in somewhere other than /usr/bin. It is annoying for syntax
highlighting, though we could work around that with some build magic.
It works fine, there's no reason to remove it.
The rehash stuff no the other hand, yeah - th
I'm 100% in favour of discarding the "exec perl" hack -- solely because
it confuses syntax highlighting something fierce :P
But I have no particular familiarity with its historical context,
so can't reliably evaluate whether it's still needed for some
obscure reason
On Fri, Jan 13, 2017, at 06:
