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".
Created attachment 18012 [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.
Created attachment 18015 [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.
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>
This patch has been pushed to master and 3.12.x.
Pushed to 3.10.x and 3.8.x will be in 3.10.6 and 3.8.13