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
Remove rakudo build dependency on shell/backticks and allow smoother windows (g|mingw32-)make spectest_smolder #2201
Comments
From @ronaldxs Currently make spectest_smolder relies on curl to send the smoke The part of the fix that uses Perl6 to remove the shell dependency is in Configure.pl.patch also has one more small needed change to fix an Running "(g|mingw32-)make spectest_smolder" under Windows is still not For now, AFAIK, the only working Windows configurations that allow for 1) You need recent version of either Strawberry Perl or ActiveState Perl BTW - I don't know of a document listing the tools needed to do this Hoping this is of help, |
From @ronaldxsbuild_Makefile.in.patchdiff --git a/build/Makefile.in b/build/Makefile.in
index d8340a5..437ab2f 100644
--- a/build/Makefile.in
+++ b/build/Makefile.in
@@ -442,7 +442,7 @@ rakudo_test_run.tar.gz: testable t/spectest.data
- $(HARNESS_WITH_FUDGE_JOBS) --tests-from-file=t/spectest.data --archive rakudo_test_run.tar.gz --parrot_revision @revision@
spectest_smolder: rakudo_test_run.tar.gz
- curl -F architecture=@cpuarch@ -F platform=@osname@ -F revision=`git log -1 --pretty=format:%H` -F report_file=@rakudo_test_run.tar.gz -F username=parrot-autobot -F password=qa_rocks http://smolder.parrot.org/app/projects/process_add_report/5
+ ./perl6 -e "run qqx[git log -1 --pretty=format:%H].fmt(qq[curl -F architecture=@cpuarch@ -F platform=@osname@ -F revision=%s -F report_file=@rakudo_test_run.tar.gz -F username=parrot-autobot -F password=qa_rocks http://smolder.parrot.org/app/projects/process_add_report/5])"
testable : all spectest_checkout spectest_update
|
From @ronaldxsConfigure.pl.patchdiff --git a/Configure.pl b/Configure.pl
index 23f986e..9562c7e 100644
--- a/Configure.pl
+++ b/Configure.pl
@@ -171,7 +171,8 @@ sub create_makefile {
if ($^O eq 'MSWin32') {
$maketext =~ s{/}{\\}g;
$maketext =~ s{\\\*}{\\\\*}g;
- $maketext =~ s{http:\S+}{ do {my $t = $&; $t =~ s'\\'/'g; $t} }eg;
+ $maketext =~ s{(?:git|http):\S+}{ do {my $t = $&; $t =~ s'\\'/'g; $t} }eg;
+ $maketext =~ s/.*curl.*/do {my $t = $&; $t =~ s'%'%%'g; $t}/meg;
}
if ($makefile_timing) {
|
From @ronaldxsForgot to mention ... I had an IRC disussion about this idea here |
@ronaldxs - Status changed from 'new' to 'open' |
From @moritzThanks for the patch, I've applied it. I'm leaving this ticket open for now, until we've decided what to do |
From @cokeOn Sat Oct 02 03:23:18 2010, moritz wrote:
Let's either add a Windows section to the main README, or a new I will happily apply such a patch! -- |
From @cokeOn Sat Oct 08 13:47:10 2011, coke wrote:
Added a section to the tail of INSTALL.txt |
@coke - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#78152 (status was 'resolved')
Searchable as RT78152$
The text was updated successfully, but these errors were encountered: