Ported --runtime-prefix option to winxed #208

Closed
wants to merge 2 commits into
from

Projects

None yet

3 participants

@pavelchristof
Contributor

I have moved handling of the --runtime-prefix commandline option from frontend/parrot2/main.c to frontedn/parrot2/prt0.winxed. It passes tests and works as intended:
pawel@DESKTOP ~ $ projects/parrot/parrot --runtime-prefix
/home/pawel/projects/parrot

@pavelchristof pavelchristof Moved handling of the --runtime-prefix commandline option from
frontend/parrot2/main.c to frontedn/parrot2/prt0.winxed.
7e5ab93
@leto
Member
leto commented Nov 27, 2011

Taking a look at this...

@leto
Member
leto commented Nov 27, 2011

This looks good, but if fails a few coding standard tests: http://smolder.parrot.org/app/projects/report_details/24597

I can merge this if you fix those up. You can generate a similar report with "make smolder_fulltest"

@pavelchristof
Contributor

I have fixed them up. You can see smolder report here: http://smolder.parrot.org/app/projects/report_details/24603

@Whiteknight
Contributor

@leto: Actually, I just ran the tests myself and don't see any codetest failures. I think there were unrelated failures in master that were clouding the results. Those are fixed as of this morning, and an updated pull tests clean. I'm going to merge this soon, I thik

@pavelchristof
Contributor

Yes, they were unrelated to my commit.

@Whiteknight
Contributor

I've pulled these changes into the runtime_prefix branch for further testing. Thanks!

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