Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PATCH] Show how to work around the <5.9.5 makedepend.SH syntax error #10294

Closed
p5pRT opened this issue Apr 9, 2010 · 4 comments
Closed

[PATCH] Show how to work around the <5.9.5 makedepend.SH syntax error #10294

p5pRT opened this issue Apr 9, 2010 · 4 comments

Comments

@p5pRT
Copy link

p5pRT commented Apr 9, 2010

Migrated from rt.perl.org#74192 (status was 'resolved')

Searchable as RT74192$

@p5pRT
Copy link
Author

p5pRT commented Apr 9, 2010

From @avar

From​: Ævar Arnfjörð Bjarmason <avar@​cpan.org>

Before 96a8704 makedepend.SH used shell quote syntax that modern
shells barf on. Document how to work around this in the example bisect
script.


pod/perlrepository.pod | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)

Inline Patch
diff --git a/pod/perlrepository.pod b/pod/perlrepository.pod
index 65a9bef..731735d 100644
--- a/pod/perlrepository.pod
+++ b/pod/perlrepository.pod
@@ -645,6 +645,14 @@ testcase:
   % cat ~/run
   #!/bin/sh
   git clean -dxf
+
+  # If you get './makedepend: 1: Syntax error: Unterminated quoted
+  # string' when bisecting versions of perl older than 5.9.5 this hack
+  # will work around the bug in makedepend.SH which was fixed in
+  # version 96a8704c. Make sure to comment out `git co makedepend.SH'
+  # below too.
+  git show blead:makedepend.SH > makedepend.SH
+
   # If you can use ccache, add -Dcc=ccache\ gcc -Dld=gcc to the Configure line
   # if Encode is not needed for the test, you can speed up the bisect by
   # excluding it from the runs with -Dnoextensions=Encode
@@ -658,6 +666,7 @@ testcase:
   ./perl -Ilib ~/testcase.pl
   ret=$?
   [ $ret -gt 127 ] && ret=127
+  # git co makedepend.SH
   git clean -dxf
   exit $ret
 
-- 
1.7.0.4.298.gc81d

@p5pRT
Copy link
Author

p5pRT commented Apr 13, 2010

From @rgs

Thanks, applied to bleadperl

@p5pRT
Copy link
Author

p5pRT commented Apr 13, 2010

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Apr 13, 2010

@rgs - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant