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
make t/spec/foo/bar.t modifies the current installation #6037
Comments
From @toolforgerTesting an installation should not modify it, however make -n for an ~/try-out-roast/rakudo$ make -n t/spec/S22-package-format/local.t I.e. it's copying perl6-m over perl6, which it shouldn't do. |
From @zoffixznetOn Sat, 28 Jan 2017 02:10:47 -0800, toolforger@durchholz.org wrote:
I assume that's to select which backend to use for testing. `perl6` is just a 1-line wrapper bash script.
From `make -n test` I see that it doesn't. Roast contains language compliance tests, A make target can be made that just runs the roast tests. What benefits will we attain by having such an option? |
The RT System itself - Status changed from 'new' to 'open' |
From @toolforgerMy approach to solving this would be to have make test use perl-m resp. Adding another make target still leaves "make test" overwriting the |
From @zoffixznetOn Sun, 29 Jan 2017 01:56:16 -0800, jo@durchholz.org wrote:
OK. It already does so. As stated above, `make test` doesn't overwrite anything. Closing. |
@zoffixznet - Status changed from 'open' to 'rejected' |
From @toolforgerAm 29.01.2017 um 14:58 schrieb Zoffix Znet via RT:
Then it does not need to overwrite perl6.
This statement is in direct contradiction to reported observation. ~/try-out-roast/rakudo$ make -n t/spec/S22-package-format/local.t Can you explain? |
From @zoffixznetOn Sun, 29 Jan 2017 08:46:28 -0800, jo@durchholz.org wrote:
You're not running `make test` in that command. |
@zoffixznet - Status changed from 'rejected' to 'open' |
From @zoffixznetOn Sun, 29 Jan 2017 09:03:34 -0800, cpan@zoffix.com wrote:
A few data points: |
From @toolforgerAm 29.01.2017 um 18:03 schrieb Zoffix Znet via RT:
Oh. |
From @toolforgerAm 29.01.2017 um 19:28 schrieb Zoffix Znet via RT:
Thanks, I find that kind of lateral information useful. (Just for the record: I'd prefer it if roast didn't access ./perl6.)
Agreeing with observation and conclusions. Maybe the fix is as simple as removing the rm/cp/chmod commands. |
Migrated from rt.perl.org#130660 (status was 'open')
Searchable as RT130660$
The text was updated successfully, but these errors were encountered: