!VRULIdgoKmKPzJZzjj:nixos.org

Nix Hackers

876 Members
For people hacking on the Nix package manager itself187 Servers

Load older messages


SenderMessageTime
29 Aug 2025
@fzakaria:one.ems.hostfzakaria

path.emplace_back("");

23:44:52
@fzakaria:one.ems.hostfzakaria(it was pushing back forward slash before)23:45:14
@Ericson2314:matrix.orgJohn Ericsongive it a pull23:48:56
@fzakaria:one.ems.hostfzakariaoh i was gonan land them ;P23:49:11
@Ericson2314:matrix.orgJohn Ericsonoh sorry23:49:22
@Ericson2314:matrix.orgJohn EricsonI did that because the initial "" makes an initial /23:49:33
@Ericson2314:matrix.orgJohn EricsonI get a test failure now23:49:56
@Ericson2314:matrix.orgJohn Ericsonbut at least local and CI should agree23:50:00
@fzakaria:one.ems.hostfzakariafrom what ?23:51:22
@fzakaria:one.ems.hostfzakariai thin kits from the new try23:52:39
@fzakaria:one.ems.hostfzakariaparseURL actually parses scp like...23:52:50
@fzakaria:one.ems.hostfzakariathat's why i had the if guard23:52:54
@fzakaria:one.ems.hostfzakaria(it parses it wrong... but it succeeds the parse)23:53:07
@fzakaria:one.ems.hostfzakariai.e. github.com:/owner/repo.git23:53:13
@fzakaria:one.ems.hostfzakaria(probably another boost bug to open)23:53:18
@fzakaria:one.ems.hostfzakariak i added those guards back and pushed.23:55:38
@fzakaria:one.ems.hostfzakaria
    if (auto r = boost::urls::parse_uri(url); r && r->has_scheme() && r->has_authority()) {
        return parseURL(url);
    }

    if (hasPrefix(url, "file:"))
        return parseURL(url);
23:55:45
30 Aug 2025
@Ericson2314:matrix.orgJohn Ericsonwe should not use two UIR parsers00:01:46
@Ericson2314:matrix.orgJohn Ericsonif we need to check schem and authority00:01:55
@Ericson2314:matrix.orgJohn Ericsonwe can do it on our parseURL 00:01:59
@fzakaria:one.ems.hostfzakariagonna make dinner; put a comment on the PR00:02:50
@fzakaria:one.ems.hostfzakariaand i'll follow up on it00:02:52
@fzakaria:one.ems.hostfzakaria👍️00:02:56
@fzakaria:one.ems.hostfzakaria* gonna make dinner; put a comment on the PR please00:03:02
@fzakaria:one.ems.hostfzakariaI think boost and parseURL are too lenient though and scp style are making it through00:04:57
@fzakaria:one.ems.hostfzakariasaw your comment.00:05:27
@Ericson2314:matrix.orgJohn Ericsoncool00:19:23
@Ericson2314:matrix.orgJohn Ericsonand sorry I had missed your comments00:19:28
@tsn9cp_again:matrix.orgtsn9cp_againQuick question about the naming convention of package/derivation versions in Nix: If I build from a specific commit, which is present only in a specific branch, how would I name the version? AFAIK the convention for non-tagged commit on the main branch is "unstable-YYYY-MM-DD". Where would the branch fit into this?10:36:37
@k900:0upti.meK900 Better question for #Nixpkgs / NixOS contributions 10:37:11

Show newer messages


Back to Room ListRoom Version: 6