lassulus changed the topic of #nixos-de to: Willkommen im deutschen NixOS Channel.
mbrgm_ has joined #nixos-de
mbrgm has quit [Ping timeout: 272 seconds]
mbrgm_ is now known as mbrgm
h0m1 has quit [Ping timeout: 240 seconds]
h0m1 has joined #nixos-de
xkey has quit [Ping timeout: 240 seconds]
xkey has joined #nixos-de
palo1 has joined #nixos-de
palo has quit [Ping timeout: 240 seconds]
palo1 is now known as palo
xkey has quit [Remote host closed the connection]
xkey has joined #nixos-de
makefu has quit [Quit: WeeChat 2.6]
Chiliparrot has joined #nixos-de
makefu has joined #nixos-de
Chiliparrot has quit [Quit: My iMac has gone to sleep. ZZZzzz…]
Chiliparrot has joined #nixos-de
Chiliparrot has quit [Ping timeout: 240 seconds]
Chiliparrot has joined #nixos-de
das_j has joined #nixos-de
Scriptkiddi has joined #nixos-de
ajs124 has joined #nixos-de
Synthetica has joined #nixos-de
schmittlauch[m] has joined #nixos-de
<schmittlauch[m]> github-workflow Frage: Wie ist die cleverste Art, den Branch eines offenen PRs auszuchecken? `git fetch`?
<schmittlauch[m]> Und geht das auch, wenn der merge conflicts hat?
<schmittlauch[m]> tab-completion sagt `git fetch origin pulls/1337/head`
Chiliparrot has quit [Quit: My iMac has gone to sleep. ZZZzzz…]
<makefu> für nixpkgs: nix-review pr #number
<makefu> in nem ausgecheckten nixpkgs repo
CRTified has joined #nixos-de
ma27[m] has joined #nixos-de
<ma27[m]> falls du nach ner allgemeinen lösung suchst empfehle ich `hub checkout <url zum PR>` (https://github.com/github/hub/).
<schmittlauch[m]> makefu: Das ging ja gerade wegen merge conflicts nicht
<makefu> evtl git reset --hard master und nochmal versuchen? afair nimmt nix-review die ausgecheckte rev zum testen
<schmittlauch[m]> makefu: der PR hatte nen conflict gegen master
<schmittlauch[m]> makefu: kannst gerne selbst probieren: nix-review pr 75664
<schmittlauch[m]> mach jetzt nen neues PR ohne conflicts auf
<makefu> -c {merge,commit}, --checkout {merge,commit}
<makefu> da kannst du sagen auf was es drauf gemerged werden soll
<schmittlauch[m]> ah
<schmittlauch[m]> Mag mal jemand https://github.com/NixOS/nixpkgs/pull/79848 mergen? Hab das ausgegraben, getestet & rebased.
<{^_^}> #79848 (by schmittlauch, 3 minutes ago, open): tipp10: init at 3.1.0
<schmittlauch[m]> Ob ich nen Backport-PR selbst aufmachen soll hab ich immer noch nicht ganz verstanden.
<schmittlauch[m]> Mic92: ^
<makefu> keinn weiss wan gebackportet werden darf/muss/soll xD
<Mic92> makefu: Neue packete sind in der Regel ok.
<Mic92> Meistens sind die maintainer nur zu faul dafür.
<makefu> good to know :)
<schmittlauch[m]> thx
<Mic92> Bzw. manchmal will man das Packet erstmal reifen lassen in master um evtl. packaging bugs zu finden.
<lassulus> aber auf master testen so wenige
<schmittlauch[m]> worldofpeace portet selbst back, andere wollen dass ich PRs aufmache.
<lassulus> aber ich würde ja sagen, leute sollen selber backport PRs machen
<Mic92> Gefühlt gibt es viele Poweruser auf master.
<ma27[m]> schmittlauch: da das IMHO für den merger weniger Arbeit ist, mache z.B. ich auch selber backports. Da das aber tatsächlich ziemlich inkonsistent ist, empfehle ich, explizit auf den Wunsch eines Backports hinzuweisen, zumindest meiner Erfahrung nach, erledigen Maintainer dann auch häufig den Backport :)
<schmittlauch[m]> Vorteil am PR selbst machen: kann vorher getestet werden, dass es auf beiden branches läuft. Nachteil: Mehr PRs, zeitliche Abhängigkeit der PRs und von Änderungen in ihnen.
<lassulus> na den backport würde ich auch erst machen wenn der gegen master gemerged wird. aber ich versuch meist auch dei backports gleich zu machen wenn das im PR steht
<lassulus> aber wenn das nicht drin steht mach ich auch keinen
<schmittlauch[m]> lassulus: du testest den aber nicht gegen den backport branch. Problem am backport PR nach master merge machen ist die zeitliche Verzögerung, die Person die den 1. PR gemerged hat ist dann evtl wieder afk
<lassulus> na ich versuch das schon zu testen, aber mit PR geht da einfach nix-review.
<lassulus> das macht das meiner meinung nach etwas smoother
<Mic92> Wenn ofborg schon mal den commit evaluiert hat, geht das mit nix-review deutlich fixer.
<Mic92> Das ist der Vorteil beim Pull request.
<schmittlauch[m]> Ok, dann mach ich in Zukunft PRs.
<lassulus> hmm, kann man haskellPackages.niv auch ohne doc für alle abhängigkeiten installieren?
<Mic92> lassulus: oh my gosh hat das viele abhängigkeiten :(
<Mic92> Mhm. das sind aber alles bloß build dependencies.
<lassulus> jo, bin da auch iwie verwirrt
<Mic92> lassulus: evtl. ist nix-shell daran schuld
<lassulus> ( ﾟдﾟ)
<Mic92> Oder es ist gerade auf der hydra kaputt
<Mic92> und nix baut es lokal.
<Mic92> Das wird es sein.
<lassulus> oh noes
<lassulus> aber wieso brauch ich die ganzen docs? :D
<Mic92> lassulus: du musst nur härter upgraden: https://hydra.nixos.org/eval/1569985?filter=niv&compare=1569939&full=#tabs-still-succeed in master scheint es wieder ok zu sein.
<lassulus> und in 19.09?
<Mic92> b9cb3b2
<lassulus> hmm, dann muss ich vl channel bumpen
<lassulus> was ich grade eigentlich mit niv tun wollte :D
<lassulus> Mic92: er hat niv garich gebaut
<lassulus> er hat nur 200 dinge runtergeladen
Chiliparrot has joined #nixos-de
Chiliparrot has quit [Quit: My iMac has gone to sleep. ZZZzzz…]