!PbcQeaWcgMyjVfeGQN:nixos.org

Nix Mozilla 🦊🐦🐒

165 Members
A room about a number of weird animals (also known as Mozilla products): Firefox, Thunderbird, Spidermonkey, NSS, cacert. Also a little bit of fun times, small amounts of extreme, when building weird animals. But for bugs please file GitHub issues. | Release Schedule: https://whattrainisitnow.com | Crash-Stats: https://crash-stats.mozilla.org/search/?distribution_id=%3Dnixos&product=Firefox&product=Thunderbird51 Servers

Load older messages


SenderMessageTime
16 Oct 2024
@yannis:mozilla.org@yannis:mozilla.org changed their display name from yannis|pto back oct 16 to yannis.06:58:56
@artturin:matrix.orgArtturinhttps://github.com/NixOS/nixpkgs/pull/34911619:26:19
22 Oct 2024
@hexa:lossy.networkhexaI'm not sure how I feel about about the PR to add darwin support for firefox-unwrapped11:20:12
@hexa:lossy.networkhexaespecially from a relatively new maintainer11:20:40
@hexa:lossy.networkhexathere's probably little choice than given it a shot?11:21:24
@k900:0upti.meK900 ⚡️It was discussed in the Mac room a bunch the other day11:25:20
@k900:0upti.meK900 ⚡️Maybe you should talk to the Darwin people about it11:25:36
@k900:0upti.meK900 ⚡️They probably have a better idea of how annoying it'll be to maintain11:25:55
@vcunat:matrix.orgvcunatAn alternative would be to separate it.11:26:05
@vcunat:matrix.orgvcunatBecause what do we do on updates when darwin version breaks and isn't easy to fix?11:26:26
@vcunat:matrix.orgvcunatMaybe it won't ever happen that way and we could split at that point, though.11:28:01
@hexa:lossy.networkhexa * there's probably little choice than giving it a shot?11:34:32
23 Oct 2024
@luna-null:matrix.org@luna-null:matrix.org changed their display name from Autumn to luna-null.09:49:30
24 Oct 2024
@jopejoe1:matrix.orgjopejoe1 (4094@39c3) set a profile picture.07:34:43
25 Oct 2024
@lholh:matrix.orglholh joined the room.03:54:23
@chrispickard:matrix.orgchrispickard left the room.18:51:21
27 Oct 2024
@ritiek:matrix.orgritiek joined the room.23:19:15
29 Oct 2024
@vcunat:matrix.orgvcunathttps://github.com/NixOS/nixpkgs/pull/35216119:33:23
@scrumplex:duckhub.ioScrumplex joined the room.20:07:42
@scrumplex:duckhub.ioScrumplexboth -devedition and -beta fail to apply /nix/store/9lx1746yy56pn3dpc35l1xr6l3lykbin-env_var_for_system_dir-ff111.patch in the PR above20:10:38
@scrumplex:duckhub.ioScrumplexLooking at the trunk (tip?) it just seems like it was moved around in the file. I will try to update the patch for 133 onwards20:11:20
@phileas:asra.grsyd installs gentoo (they/them) joined the room.20:13:37
@hexa:lossy.networkhexaok20:17:36
@scrumplex:duckhub.ioScrumplex

Current patch versus new patch. Seems like the function signature has changed slightly:

--- pkgs/applications/networking/browsers/firefox/env_var_for_system_dir-ff111.patch	2024-08-10 21:13:22.142083987 +0200
+++ pkgs/applications/networking/browsers/firefox/env_var_for_system_dir-ff133.patch	2024-10-29 21:25:17.412737463 +0100
@@ -10,13 +10,13 @@
  #include "prprf.h"
  
  #include "nsIAppStartup.h"
-@@ -309,7 +310,8 @@ static nsresult GetSystemParentDirectory(nsIFile** aFile) {
+@@ -297,7 +297,8 @@ static nsresult GetSystemParentDirectory(nsIFile** aFile) {
        "/usr/lib/mozilla"_ns
  #    endif
        ;
--  rv = NS_NewNativeLocalFile(dirname, false, getter_AddRefs(localDir));
+-  rv = NS_NewNativeLocalFile(dirname, getter_AddRefs(localDir));
 +  const char* pathVar = PR_GetEnv("MOZ_SYSTEM_DIR");
-+  rv = NS_NewNativeLocalFile((pathVar && *pathVar) ? nsDependentCString(pathVar) : reinterpret_cast<const nsCString&>(dirname), false, getter_AddRefs(localDir));
++  rv = NS_NewNativeLocalFile((pathVar && *pathVar) ? nsDependentCString(pathVar) : reinterpret_cast<const nsCString&>(dirname), getter_AddRefs(localDir));
  #  endif
  
    if (NS_SUCCEEDED(rv)) {
20:26:49
@scrumplex:duckhub.ioScrumplexThis new patch applied successfully. Let's see if it will build20:27:55
@scrumplex:duckhub.ioScrumplex

Nope :(

       >  1:02.52(B checking for nss >= 3.106... no(B(B
       >  1:02.52(B ERROR: Requested 'nss >= 3.106' but version of NSS is 3.105.0(B(B
20:29:39
@hexa:lossy.networkhexaon staging20:29:44
@scrumplex:duckhub.ioScrumplexI see. Sooo rebase the PR on staging?20:30:00
@hexa:lossy.networkhexabecause ... reasons20:30:02
@hexa:lossy.networkhexalet me check20:30:14

Show newer messages


Back to Room ListRoom Version: 9