!rGlCMuXgAhgEpdvJUz:nixos.org

NixOS KDE

206 Members
56 Servers

Load older messages


SenderMessageTime
27 Aug 2023
@nadimkobeissi:matrix.orgNadim Kobeissi It doesn't seem to have recompiled plasma-desktop while creating the ISO 08:48:41
@k900:0upti.meK900 ⚡️ Does <nixpkgs> point at your modified nixpkgs checkout? 08:51:06
@nadimkobeissi:matrix.orgNadim KobeissiNo, I don't think so08:51:23
@k900:0upti.meK900 ⚡️ Then you want to use ~/Code/nixos/nixpkgs/nixos instead of <nixpkgs/nixos> 08:51:42
@nadimkobeissi:matrix.orgNadim KobeissiUnderstood08:51:50
@nadimkobeissi:matrix.orgNadim Kobeissi NIX_PATH=localdev=/home/nadim/Code/nixos/nixpkgs/nixos nix-build '<localdev>' -A config.system.build.isoImage -I nixos-config=installation-cd-graphical-plasma5.nix 08:55:41
@nadimkobeissi:matrix.orgNadim KobeissiScreenshot_20230827_110456.png
Download Screenshot_20230827_110456.png
09:05:29
@nadimkobeissi:matrix.orgNadim KobeissiIt works!09:05:30
@nadimkobeissi:matrix.orgNadim Kobeissi K900 ⚡️: Updated the issue: https://github.com/NixOS/nixpkgs/pull/251726 09:11:46
@k900:0upti.meK900 ⚡️As I said, please squash your commits and make the commit message match the guidelines in CONTRIBUTING.md09:12:09
@k900:0upti.meK900 ⚡️Also please include these screenshots in the issue09:12:22
@nadimkobeissi:matrix.orgNadim KobeissiThe commit message does match, no?09:12:30
@k900:0upti.meK900 ⚡️There's three, currently, and no, they don't09:14:07
@k900:0upti.meK900 ⚡️Also, I'm pretty sure you can remove your Kickoff changes09:14:31
@nadimkobeissi:matrix.orgNadim KobeissiWhy, though?09:14:42
@nadimkobeissi:matrix.orgNadim KobeissiWhy do we want Discover in kickoff?09:14:54
@k900:0upti.meK900 ⚡️We dpm09:16:01
@k900:0upti.meK900 ⚡️ * We don't09:16:02
@k900:0upti.meK900 ⚡️But Kickoff should automatically ignore non-existant shortcuts09:16:11
@nadimkobeissi:matrix.orgNadim KobeissiAlright09:16:17
@k900:0upti.meK900 ⚡️Also, please rebase on top of master if you need changes from master09:17:23
@k900:0upti.meK900 ⚡️Instead of merging it09:17:25
@k900:0upti.meK900 ⚡️Otherwise the history will be very messy09:17:32
@nadimkobeissi:matrix.orgNadim KobeissiNo problem09:18:29
@nadimkobeissi:matrix.orgNadim KobeissiAlright, here we go: https://github.com/NixOS/nixpkgs/pull/25174609:32:46
@k900:0upti.meK900 ⚡️Would prefer the commit message to be something more descriptive09:37:29
@k900:0upti.meK900 ⚡️Like "plasma-desktop: remove Discover shortcut from default panel"09:37:42
@nadimkobeissi:matrix.orgNadim KobeissiI think we're good, man. :P09:37:43
@k900:0upti.meK900 ⚡️We're really not09:38:56
@k900:0upti.meK900 ⚡️This is a huge repo with thousands of contributors09:39:02

There are no newer messages yet.


Back to Room ListRoom Version: 9