infinisil has quit [(Quit: ZNC 1.6.5 - http://znc.in)]
mbrgm has quit [(Ping timeout: 248 seconds)]
mbrgm has joined joined #nixos-dev
taktoa has joined joined #nixos-dev
JosW has joined joined #nixos-dev
orivej has quit [(Ping timeout: 240 seconds)]
ckauhaus has joined joined #nixos-dev
goibhniu has joined joined #nixos-dev
__Sander__ has joined joined #nixos-dev
<MoreTea>
gchristensen: as an addition to my remark to just use a spreadsheet for security triageing, there is a python lib that can read spreadsheets. http://tinaja.computer/2017/10/27/gspread.html
<gchristensen>
:o
<MoreTea>
:%s/spreasheets/google spreadsheets/
<MoreTea>
We could write a simple python IRC bot that read that file and pings security triage peepz :)
<Profpatsch>
Okay, I’m not sure why this was a thing in the first place then.
<Profpatsch>
thx
<Profpatsch>
grahamc: Do we need a rebuild-x:0 tag?
<Profpatsch>
It may be a bit noisy on each PR.
<Profpatsch>
But very awesome that this works!!
<Profpatsch>
Is it updated as master moves?
<Profpatsch>
grahamc: Do you think it’s possible to get something like mention-bot running again?
<Profpatsch>
I really miss that.
<grahamc>
Yeah I’m not sure. It is possibly too noisy. It isn’t updated when master moves. Just when the pr is updated. The 0 tag was to indicate it has finished looking at the PR (before you couldn’t know if it looked or was 0)
<Profpatsch>
PR update should work, too. Maybe add a batch job that regularly updates all stale PRs?
<Dezgeg>
well I doubt the amount of rebuilds usually changes that much even if master moves
<Profpatsch>
grahamc: Is it possible to integrate it into the PR like e.g. Travis is? With an up-to-date thingy at the bottom of the PR?
<Profpatsch>
Dezgeg: You might be right.
<Dezgeg>
it is actually there already in some form: grahamcofborg-eval — Evaluation checks OK
<Profpatsch>
Hm, is there actually a way for the amount of packages affected to go up (apart from new packages)?
<Profpatsch>
Maybe it really doesn’t matter at all.
<Dezgeg>
if someone pushes to master something adds that package as a dependency of some other huge package
<Profpatsch>
yeah, right.
<Profpatsch>
Maybe batch-check stuff once a week?
<Profpatsch>
And move the “was checked by bot” tag to the bottom of the PR, this way the 0–10 tags could be removed.
<Profpatsch>
If one of those deps is Chromium or Firefox, that’s still of concern of course.
<Profpatsch>
phew, I think the int.between PR is finally ready for merge.
<Profpatsch>
Next up would be restricting all the options to more specific types.
<Profpatsch>
At least to have some nice examples.
<grahamc>
Profpatsch I could add a rebuild count to the build success / fail result, sure, and drop the 0 tag and 1-10 tag,,, but although seeing either of those tags means it is easy.
<grahamc>
Or easier to merge
<Profpatsch>
Also true!
<Profpatsch>
Nice, actually.
<Profpatsch>
So scrap that, I like that.
<grahamc>
Yeah. Okay :) we’ll try it a while and then change if we still don’t like it
orivej has joined joined #nixos-dev
orivej has quit [(Ping timeout: 240 seconds)]
<MoreTea>
Profpatsch: I've got code that will extract the maintainers from pkgs/*.
<MoreTea>
I wanted to wait with a mentionbot until the maintainers RFC was done (then nixbot could @-mention as well), but based on that expression I could whip up a simple bot
<LnL>
that would be nice, I miss the mention bot
<pierron>
Profpatsch: I will try to review the int types tonight or tomorrow.
<pierron>
grahamc: I like the rebuild amount reports. Still, I am not sure about the colors, as today's colors are matching the prefix numbers.
taktoa has quit [(Remote host closed the connection)]
<Profpatsch>
pierron: thanks
<domenkozar>
pierron: can overlays specify lib?
<domenkozar>
so that it's overridden for lib argument in nixos moduels as well?
lo1tuma has joined joined #nixos-dev
<pierron>
domenkozar: use pkgs.lib, the lib given as argument of the modules are made for the module system.
<pierron>
domenkozar: another option would be for you to do: _modules.args.myLib = pkgs.lib;
<pierron>
domenkozar: which will add the myLib argument to every module.
<domenkozar>
uh, so many indirections
<pierron>
domenkozar: the problem you see is that NixOS is asking for a specific Nixpkgs version, and the lib is the version of the Module system, which happen to be loaded before.
<pierron>
domenkozar: we have no garantee over the lib that we will get from the Nixpkgs version that you are loading.
<aszlig>
domenkozar: another way would be to call the module system with your custom lib
<orivej>
Is there something like `nix why-depends nixpkgs.pkg nixpkgs.dep` that can tell why the build of pkg depends on dep? (nix why-depends tells this about the runtime closure, and hence needs to build the pkg to answer this query)
<vcunat>
jtojnar: right now Hydra's slightly overloaded, but I'll do it within a few days
<vcunat>
the branch isn't missing too many commits from current master or staging, right?
<Mic92>
copumpkin: it is not that interesting actually. Just a private repos, where people ask questions about how to deal with open-source contributions.