Bug 10156 - Fix usr/bin/perl path for get_report_social_data.pl
Summary: Fix usr/bin/perl path for get_report_social_data.pl
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: David Cook
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-29 23:54 UTC by David Cook
Modified: 2013-12-05 20:09 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 10156 - Fix usr/bin/perl path for get_report_social_data.pl (1.04 KB, patch)
2013-05-08 08:09 UTC, David Cook
Details | Diff | Splinter Review
Bug 10156 - Fix usr/bin/perl path for get_report_social_data.pl (1.26 KB, patch)
2013-05-08 08:53 UTC, Magnus Enger
Details | Diff | Splinter Review
Bug 10156 - Fix usr/bin/perl path for get_report_social_data.pl (1.36 KB, patch)
2013-05-08 11:49 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description David Cook 2013-04-29 23:54:58 UTC
Currently, get_report_social_data.pl begins with "#!/bin/perl" rather than "#!/usr/bin/perl". 

While perl -c says the syntax is fine, it's been shown in one Debian install that the file won't work "zsh: no such file or directory: /bin/perl".
Comment 1 David Cook 2013-05-08 08:09:52 UTC Comment hidden (obsolete)
Comment 2 Magnus Enger 2013-05-08 08:53:12 UTC Comment hidden (obsolete)
Comment 3 Marcel de Rooy 2013-05-08 11:49:29 UTC
Created attachment 18019 [details] [review]
Bug 10156 - Fix usr/bin/perl path for get_report_social_data.pl

Currently, the perl path for get_report_social_data.pl is "#!/bin/perl"
when it should be "#!/usr/bin/perl".

While I'm not entirely sure how to test this, rangi did mention:

23:47 (it wouldnt work on my debian install)
23:48  zsh: no such file or directory: /bin/perl

It's a pretty trivial change, so I'm not sure it needs much of a test
plan.

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Before the patch "grep -r '#!/bin/perl' in the misc dir shows one file
with this line, and it is indeed get_report_social_data.pl. After the
patch there are no occurrences left.
Passed-QA-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Jared Camins-Esakov 2013-05-08 13:52:41 UTC
This patch has been pushed to master and 3.12.x.
Comment 5 Chris Cormack 2013-05-12 03:15:03 UTC
Pushed to 3.10.x and 3.8.x will be in 3.10.6 and 3.8.13