Skip to content

Commit 9df4879

Browse files
rhendriczeorin
andcommitted
firefox-{beta,devedition}: make binaryName match desktop file
This branches off of the unmerged PR #294971, which had the goal of distinguishing between the various Firefox packages by giving them all unique binaryNames. The goal of this PR, by contrast, is making the names of the XDG desktop files that we generate match the name by which Firefox will identify itself over D-Bus. Co-authored-by: Xandor Schiefer <[email protected]>
1 parent cc84526 commit 9df4879

File tree

6 files changed

+5
-10
lines changed

6 files changed

+5
-10
lines changed

pkgs/applications/networking/browsers/firefox/common.nix

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -425,6 +425,7 @@ buildStdenv.mkDerivation {
425425
"--disable-updater"
426426
"--enable-application=${application}"
427427
"--enable-default-toolkit=${toolkit}"
428+
"--with-app-name=${binaryName}"
428429
"--with-distribution-id=org.nixos"
429430
"--with-libclang-path=${lib.getLib llvmPackagesBuildBuild.libclang}/lib"
430431
"--with-wasi-sysroot=${wasiSysRoot}"

pkgs/applications/networking/browsers/firefox/packages/firefox-beta.nix

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
buildMozillaMach rec {
1111
pname = "firefox-beta";
12+
binaryName = pname;
1213
version = "135.0b9";
1314
applicationName = "Mozilla Firefox Beta";
1415
src = fetchurl {
@@ -27,7 +28,7 @@ buildMozillaMach rec {
2728
# not in `badPlatforms` because cross-compilation on 64-bit machine might work.
2829
maxSilent = 14400; # 4h, double the default of 7200s (c.f. #129212, #129115)
2930
license = lib.licenses.mpl20;
30-
mainProgram = "firefox";
31+
mainProgram = binaryName;
3132
};
3233
tests = {
3334
inherit (nixosTests) firefox-beta;

pkgs/applications/networking/browsers/firefox/packages/firefox-devedition.nix

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
buildMozillaMach rec {
1111
pname = "firefox-devedition";
12+
binaryName = pname;
1213
version = "135.0b9";
1314
applicationName = "Mozilla Firefox Developer Edition";
1415
requireSigning = false;
@@ -29,7 +30,7 @@ buildMozillaMach rec {
2930
# not in `badPlatforms` because cross-compilation on 64-bit machine might work.
3031
maxSilent = 14400; # 4h, double the default of 7200s (c.f. #129212, #129115)
3132
license = lib.licenses.mpl20;
32-
mainProgram = "firefox";
33+
mainProgram = binaryName;
3334
};
3435
tests = {
3536
inherit (nixosTests) firefox-devedition;

pkgs/applications/networking/browsers/floorp/default.nix

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@
2828
};
2929

3030
extraConfigureFlags = [
31-
"--with-app-name=${pname}"
3231
"--with-app-basename=${applicationName}"
3332
"--with-unsigned-addon-scopes=app,system"
3433
"--enable-proxy-bypass-protection"

pkgs/applications/networking/browsers/librewolf/librewolf.nix

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ rec {
99
extraPatches = [ "${source}/patches/pref-pane/pref-pane-small.patch" ];
1010

1111
extraConfigureFlags = [
12-
"--with-app-name=librewolf"
1312
"--with-unsigned-addon-scopes=app,system"
1413
];
1514

pkgs/top-level/all-packages.nix

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13457,16 +13457,10 @@ with pkgs;
1345713457

1345813458
firefox = wrapFirefox firefox-unwrapped { };
1345913459
firefox-beta = wrapFirefox firefox-beta-unwrapped {
13460-
nameSuffix = "-beta";
1346113460
desktopName = "Firefox Beta";
13462-
wmClass = "firefox-beta";
13463-
icon = "firefox-beta";
1346413461
};
1346513462
firefox-devedition = wrapFirefox firefox-devedition-unwrapped {
13466-
nameSuffix = "-devedition";
1346713463
desktopName = "Firefox Developer Edition";
13468-
wmClass = "firefox-devedition";
13469-
icon = "firefox-devedition";
1347013464
};
1347113465

1347213466
firefox-mobile = callPackage ../applications/networking/browsers/firefox/mobile-config.nix { };

0 commit comments

Comments
 (0)