!OqhvaDMJdKYUicLDiE:nixos.org

Nixpkgs Stdenv

233 Members
75 Servers

Load older messages


SenderMessageTime
8 Mar 2023
@winterqt:nixos.devWinter (she/her)
In reply to @trofi:matrix.org
Looks like it's intentional after all: It's added by ...-glibc-2.35-224/lib/libc.so: GROUP ( ...-glibc-2.35-224/lib/libc.so.6 ...-glibc-2.35-224/lib/libc_ nonshared.a AS_NEEDED ( ...-glibc-2.35-224/lib/ld-linux-aarch64.so.1 ) ) and provides __stack_chk_guard@@GLIBC_2.17 symbol.
* Where did you find this info?
02:53:19
@vcunat:matrix.orgvcunat
In reply to @winterqt:nixos.dev
Where did you find this info?
That's the libc.so text-file.
07:23:25
@trofi:matrix.orgtrofi
In reply to @winterqt:nixos.dev
Where did you find this info?

Ran

$ nix develop -f. hello --argstr system aarch64-linux
$ unpackPhase; cd hello*; patchPase; configurePhase; buildPhase

Then explore unstripped/unpatchelf-ed hello and realized it has the dependency.
Then explored gcc -o hello ... command, added -v to it and extracted ld call. That still produced a reference.
Then added -shared flag to allow removing almost all .o files and noticed that at some point ld.so NEEDED dependency disappears. That means it was an explicit linkage input and some symbol is retaining it.
Then ran nm -D on ld.so and on hello to see if there are suspects that are exported from ld.so and are imported to hello. __stack_chk_guard looked obvious.
Then checked all the inputs if they are linker scripts. libc.so ended up one.

07:30:07
@trofi:matrix.orgtrofi
In reply to @winterqt:nixos.dev
Where did you find this info?
*

Ran

$ nix develop -f. hello --argstr system aarch64-linux
$ unpackPhase; cd hello*; patchPase; configurePhase; buildPhase

Then explored unstripped/unpatchelf-ed hello and realized it has the dependency.
Then explored gcc -o hello ... command, added -v to it and extracted ld call. That still produced a reference.
Then added -shared flag to allow removing almost all .o files and noticed that at some point ld.so NEEDED dependency disappears. That means it was an explicit linkage input and some symbol is retaining it.
Then ran nm -D on ld.so and on hello to see if there are suspects that are exported from ld.so and are imported to hello. __stack_chk_guard looked obvious.
Then checked all the inputs if they are linker scripts. libc.so ended up one.

07:36:32
@vcunat:matrix.orgvcunat
In reply to @winterqt:nixos.dev
Where did you find this info?
* That's the libc.so text-file.
08:09:19
14 Mar 2023
@ckie:ckie.devckie (they/them) changed their display name from ckie (they/them) to ckie (they/them; heavily limited keyboard usage, dictation or voice only).01:09:43
18 Mar 2023
@rick:matrix.ciphernetics.nlMindavi changed their display name from Rick (Mindavi) to Mindavi.10:48:18
20 Mar 2023
@hexa:lossy.networkhexa
❯ nix build -f . curl.tests
error:
       … while calling a functor (an attribute set with a '__functor' attribute)

         at /home/hexa/git/nixos/staging/pkgs/tools/networking/curl/default.nix:171:20:

          170|       withCheck = finalAttrs.finalPackage.overrideAttrs (_: { doCheck = true; });
          171|       fetchpatch = tests.fetchpatch.simple.override { fetchpatch = fetchpatch.override { fetchurl = useThisCurl fetchurl; }; };
             |                    ^
          172|       curlpp = useThisCurl curlpp;

       … while evaluating a branch condition

         at /home/hexa/git/nixos/staging/lib/customisation.nix:92:7:

           91|     in
           92|       if builtins.isAttrs result then
             |       ^
           93|         result // {

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: attribute 'version' missing

       at /home/hexa/git/nixos/staging/pkgs/build-support/fetchpatch/tests.nix:4:19:

            3| let
            4|   isFetchpatch2 = fetchpatch.version == 2;
             |                   ^
            5| in

not sure if the fetchers are stdenv material, but looks like fetchpatch2 changes have broken curls passthru tests 🙂 cc Artturin

15:07:53
@artturin:matrix.orgArtturinthe override clears the // { version ... } done on the callPackage result15:18:43
@artturin:matrix.orgArtturin
  fetchpatch = callPackage ../build-support/fetchpatch {
    # 0.3.4 would change hashes: https://github.com/NixOS/nixpkgs/issues/25154
    patchutils = buildPackages.patchutils_0_3_3;
  } // {
    tests = pkgs.tests.fetchpatch;
    version = 1;
  };

  fetchpatch2 = callPackage ../build-support/fetchpatch {
    patchutils = buildPackages.patchutils_0_4_2;
  } // {
    tests = pkgs.tests.fetchpatch2;
    version = 2;
  };
15:18:47
@artturin:matrix.orgArtturinpassthru is not used here because fetchpatch retunrs a function15:19:10
@artturin:matrix.orgArtturin

one solution

diff --git a/pkgs/tools/networking/curl/default.nix b/pkgs/tools/networking/curl/default.nix
index fca190bde4a..b95275327c2 100644
--- a/pkgs/tools/networking/curl/default.nix
+++ b/pkgs/tools/networking/curl/default.nix
@@ -168,7 +168,7 @@ stdenv.mkDerivation (finalAttrs: {
     inherit opensslSupport openssl;
     tests = {
       withCheck = finalAttrs.finalPackage.overrideAttrs (_: { doCheck = true; });
-      fetchpatch = tests.fetchpatch.simple.override { fetchpatch = fetchpatch.override { fetchurl = useThisCurl fetchurl; }; };
+      fetchpatch = tests.fetchpatch.simple.override { fetchpatch = (fetchpatch.override { fetchurl = useThisCurl fetchurl; }) // { version = 1; }; };
       curlpp = useThisCurl curlpp;
       coeurl = useThisCurl coeurl;
       haskell-curl = useThisCurl haskellPackages.curl;
15:20:15
@hexa:lossy.networkhexaI'll take that, thanks15:29:21
@artturin:matrix.orgArtturinhttps://github.com/NixOS/nixpkgs/pull/222215 doesn't fix the issue either, don't know if there's a way to preserve it through .override15:38:54
@hexa:lossy.networkhexapushed the "one solution" one top of https://github.com/NixOS/nixpkgs/pull/22217215:51:33
@hexa:lossy.networkhexaw/e you prefer15:51:51
@hexa:lossy.networkhexa * pushed the "one solution" one on top of https://github.com/NixOS/nixpkgs/pull/22217216:31:36
23 Mar 2023
@ckie:ckie.devckie (they/them) changed their display name from ckie (they/them; heavily limited keyboard usage, dictation or voice only) to ckie (they/them; limited keyboard usage, voice preferred).02:03:47
24 Mar 2023
@winterqt:nixos.devWinter (she/her)https://bugs.gnu.org/6240300:42:40
@winterqt:nixos.devWinter (she/her)not sure if we bumped coreutils yet (sorry), but apply ^ when we do00:42:57
@winterqt:nixos.devWinter (she/her)see also https://bugs.archlinux.org/task/7796900:43:09
@hexa:lossy.networkhexahttps://bugs.gnu.org/6240301:21:03
@hexa:lossy.networkhexawe did not yet01:21:20
@hexa:lossy.networkhexahttps://github.com/NixOS/nixpkgs/pull/222314#issuecomment-148204469201:22:06
@winterqt:nixos.devWinter (she/her)
In reply to@hexa:lossy.network
https://bugs.gnu.org/62403
is that link different from mine? :D
01:50:27
@hexa:lossy.networkhexashut up? 😛01:50:59
@hexa:lossy.networkhexaI'm old, deal with it01:51:08
@sandro:supersandro.deSandro 🐧Do coreutils even have tests?13:51:24
@andreas.schraegle:helsinki-systems.de@andreas.schraegle:helsinki-systems.dethey have users14:02:11
@qyliss:fairydust.spaceAlyssa Rossthey have lots of tests14:08:10

Show newer messages


Back to Room ListRoom Version: 9