|
Fire is step THREE! | github.com/perl6/toolchain-bikeshed | Channel logs: irclog.perlgeek.de/perl6-toolchain/today | useful prior art: metacpan.org/pod/CPAN::Meta::Spec Set by moderator on 21 July 2017. |
|||
|
00:13
b2gills joined
01:49
ilbot3 joined
|
|||
| moderator | Fire is step THREE! | github.com/perl6/toolchain-bikeshed | Channel logs: irclog.perlgeek.de/perl6-toolchain/today | useful prior art: metacpan.org/pod/CPAN::Meta::Spec | ||
| japhb | JSON::Class won't install for me with a fresh build of everything, apparently because it installs JSON::Marshal:ver('0.012') as a dependency, but then doesn't see that as a match when JSON::Class wants JSON::Marshal:ver<0.0.7..*> . | 03:28 | |
| (I'm using zef at HEAD) | 03:29 | ||
| ugexe | japhb: github.com/ugexe/zef/issues/157 | 03:31 | |
| now that I know its showing up i'll try to get around to resolving it | |||
| but that dependency should be written as :ver<0.0.7+> | 03:32 | ||
| japhb | What about rakudo 247fc6499d ? Does that make a difference to what the fixed :ver should look like? | 03:33 | |
| ugexe | i dont see JSON::Class requiring a specific version of JSON::Marshall | ||
| oh i see what you're saying, i read it wrong | 03:34 | ||
| well, maybe not. where is JSON::Marshal:ver<0.0.7..*> at? | 03:35 | ||
| japhb | Yeah, just did `zef look JSON::Class; less lib/JSON/Class.pm` -- sure enough, line 77: use JSON::Marshal:ver(v0.0.7..*); | 03:36 | |
| ugexe | well *thats* ok | ||
| japhb | So zef is reporting the request as a string, when it was really specified as a range | ||
| And yet it didn't match the version specified by JSON::Marshal. | |||
| ugexe | zef doesn't look inside modules, only inside the meta data | ||
| japhb | JSON-Class/META6.json only specifies a dependency on JSON::Marshal, with no version at all, so it must be precomp complaining about line 77, no? | 03:38 | |
| ugexe | for me its dying on JSON::Fast:ver<0.4..*> | 03:39 | |
| japhb | ugexe: as a side effect of my rebuild-the-world script, JSON::Fast is already installed when it attempts to install JSON::Class. | 03:42 | |
| ugexe | the only place I can find 0.4..* is in pod | ||
| github.com/jonathanstowe/JSON-Mars...hal.pm#L62 | |||
| oh nm, thats =end pod :x | 03:43 | ||
| japhb | That's not in the Pod. | ||
| Yeah, jonathanstowe likes to put docs at top-of-file | |||
| ugexe | that shuld be v0.4+ or 0.4 .. * | ||
| japhb | That's going to be a gotcha, methinks | 03:44 | |
| ugexe | its a version literal, so you dont get to start a range with it without a space | 03:46 | |
| e.g. v0.4 is not the same as 0.4 | 03:47 | ||
| japhb | Ah, and the parser tries to grab more '.' with higher precedence than recognizing a range. | ||
| (I would have suspected otherwise, since '..' wouldn't normally occur inside a range, no? | 03:48 | ||
| er, inside a version literal | |||
| ugexe | the fact that it worked before is pretty bad though | ||
| today I made a PR that changed selectin from Version.new($some-version) ~~ $some-string to a proper Version.new($some-version) ~~ Version.new($some-string) | 03:49 | ||
| japhb | Was that 247fc6499d ? | 03:50 | |
| ugexe | yes | ||
| japhb wonders how many ecosystem modules are going to fail to install because they only accidentally worked before. | 03:51 | ||
| ugexe | I guess we'll see. I could revert, but I'm not sure thats a good idea because while it allows it to compile its *not* matching the way anyone would expect | 03:52 | |
| japhb | Can we detect that particular problem in e.g. Zoffix's smoke of the ecosystem? | 03:53 | |
| ugexe | somehow yeah... there is an exception type for unsatisfied dependency | 03:54 | |
| and rakudo allows you to give it a JSON exception handler, so that could be specialized for the task | 03:55 | ||
| japhb | Ah, interesting | 03:56 | |
| ugexe | m: say Version.new(v0.4..*).perl # for brevity | 03:58 | |
| camelia | v0.4.* | ||
| japhb | Collapses .. to ., interesting | 03:59 | |
| ugexe | probably splits on . and rejects empty parts | ||
| m: say Version.new(v0.4..*).parts | 04:00 | ||
| camelia | (0 4 *) | ||
| japhb | Yeah, looks like it | 04:01 | |
| ugexe | sent PRs for both modules | 04:19 | |
| japhb | thx | 04:46 | |
|
06:56
domidumont joined
07:03
domidumont joined
11:38
lizmat joined
12:30
lizmat joined
12:31
lizmat_ joined
13:53
lizmat joined
14:09
lizmat joined
14:14
domidumont joined
16:12
lizmat joined
16:53
lizmat joined
18:50
ilbot3 joined
|
|||
| moderator | Fire is step THREE! | github.com/perl6/toolchain-bikeshed | Channel logs: irclog.perlgeek.de/perl6-toolchain/today | useful prior art: metacpan.org/pod/CPAN::Meta::Spec | ||
|
19:29
lizmat joined
22:30
lizmat joined
23:00
lizmat joined
|
|||