Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Sub-Uplevel CPAN distribution.

Report information
The Basics
Id: 31403
Status: resolved
Priority: 0/
Queue: Sub-Uplevel

People
Owner: Nobody in particular
Requestors: dam [...] modsoftsys.com
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 0.18
Fixed in: (no value)



Subject: Example missing #!/usr/bin/perl
Download (untitled) / with headers
text/plain 296b
Hi, There's a missing "shebang" at the top of examples/uplevel-demo.pl . As the file is shipped with executable bit set, it seems that the intention is for it to be directly executable, which is not possible without "#!/usr/bin/perl" on the first line. Trivial patch attached. Thanks, dam
Subject: examples-shebang.patch
Index: libsub-uplevel-perl/examples/uplevel-demo.pl =================================================================== --- libsub-uplevel-perl.orig/examples/uplevel-demo.pl 2007-12-12 13:01:53.000000000 +0200 +++ libsub-uplevel-perl/examples/uplevel-demo.pl 2007-12-12 13:02:02.000000000 +0200 @@ -1,3 +1,4 @@ +#!/usr/bin/perl use strict; use warnings;
Subject: Re: [rt.cpan.org #31403] Example missing #!/usr/bin/perl
Date: Wed, 12 Dec 2007 08:02:29 -0500
To: bug-Sub-Uplevel [...] rt.cpan.org
From: "David Golden" <dagolden [...] cpan.org>
Download (untitled) / with headers
text/plain 507b
On Dec 12, 2007 6:08 AM, Damyan Ivanov via RT <bug-Sub-Uplevel@rt.cpan.org> wrote: Show quoted text
> As the file is shipped with executable bit set, it seems that the > intention is for it to be directly executable, which is not possible > without "#!/usr/bin/perl" on the first line.
Thank you for noticing that -- that's very subtle. I use several custom compiled versions of perl that don't live in /usr/bin, so I'll either add my usual incantation "#!/usr/bin/env perl" or else just remove the executable bit. David
Subject: Re: [rt.cpan.org #31403] Example missing #!/usr/bin/perl
Date: Wed, 12 Dec 2007 05:15:43 -0800
To: bug-Sub-Uplevel [...] rt.cpan.org
From: Michael G Schwern <schwern [...] pobox.com>
Download (untitled) / with headers
text/plain 792b
Damyan Ivanov via RT wrote: Show quoted text
> There's a missing "shebang" at the top of examples/uplevel-demo.pl . > > As the file is shipped with executable bit set, it seems that the > intention is for it to be directly executable, which is not possible > without "#!/usr/bin/perl" on the first line. > > Trivial patch attached.
That's not really viable as it assumes /usr/bin/perl is the location of the perl which installed Sub::Uplevel which is often untrue. There are facilities to write the correct #! line for programs to be installed, but uninstalled examples have to be run through perl. Presumably the person looking inside the source directory at the example code knows how to do this. So the executable bit should just be removed. -- Whip me, beat me, make my code compatible with VMS!
Subject: Re: [rt.cpan.org #31403] Example missing #!/usr/bin/perl
Date: Wed, 12 Dec 2007 15:33:26 +0200
To: DAGOLDEN via RT <bug-Sub-Uplevel [...] rt.cpan.org>
From: Damyan Ivanov <dam [...] modsoftsys.com>
Download (untitled) / with headers
text/plain 508b
-=| DAGOLDEN via RT, Wed, Dec 12, 2007 at 08:03:10AM -0500 |=- Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=31403 > > I use several > custom compiled versions of perl that don't live in /usr/bin, so I'll > either add my usual incantation "#!/usr/bin/env perl" or else just > remove the executable bit.
Removing executable bit (and not providing a shebang line) seems fine to me. It is just an example anyway so not so important to be run "easily". -- dam JabberID: dam@jabber.minus273.org
Download signature.asc
application/pgp-signature 189b

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #31403] Example missing #!/usr/bin/perl
Date: Wed, 12 Dec 2007 08:36:59 -0500
To: bug-Sub-Uplevel [...] rt.cpan.org
From: "David Golden" <dagolden [...] cpan.org>
Download (untitled) / with headers
text/plain 368b
On Dec 12, 2007 8:33 AM, Damyan Ivanov via RT <bug-Sub-Uplevel@rt.cpan.org> wrote: Show quoted text
> Removing executable bit (and not providing a shebang line) seems fine to > me. It is just an example anyway so not so important to be run "easily".
It's fixed in the repository. In the next release of Sub::Uplevel (whenever that will be) the executable bit will be cleared. David
fixed


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.