Vadim Belman
Vadim Belman
Sorry for the confusion. I overlooked the line which actually skips if the special file is not found. Looks good to me now. @niner are you ok with it now?
What harm does it do to have them in there? The general idea of not breaking the examples published at one of the greatest sources of code snippets appeals to...
@thundergnat Thanks for pointing at @SqrtNegInf project. I asked if he can turn his project into a testable module we can then test with Blin.
It seems to me that the correct code in `HLL/Backend.nqp` should be: ```raku my $temppath := nqp::getcomp("nqp").config() ~ '/profiler/template.html'; ```
Ok, I see. I didn't know that rakudo uses profiling code from nqp. `nqp::getcomp("nqp")` makes sense under nqp runtime only. For rakudo the line would look like: ```raku nqp::getcomp("Raku").config ~...
@lizmat the first thought. But what if somebody writes another nqp-based compiler? Actually, as I look into it, the place where template.html gets installed is incorrect because the file is...
So far, it looks like the proper solution would be to add compiler parameter to `run_profiled`, `dump_profile_data`, and `dump_instrumented_profile_data` methods. Then instead of `nqp::getcomp(...).config` `$compiler.config` can be used. Then, there...
No, it is purely our implementation detail thing. It is only affected by _Configure.pl_ command line. If there is Changing it to `nqp-home` would also go in line with `source-digest`...
@lizmat I disagree. By this logic we should use uppercase too. As well as `rakudo-home` key. Snake case would make it inconsistent with other keys in the config. It's even...
@SirBogman Thank you for submitting this PR! Could you, please, open an issue on this in https://github.com/rakudo/rakudo/issues? This would raise a flag for the need to possibly fix the issue.