Unverified Commit 82dc8376 authored by Aleksana's avatar Aleksana Committed by GitHub
Browse files

pesign: fix build with gcc14, modernize (#395125)

parents 9756c0de 34dbfa42

pkgs/by-name/pe/pesign/package.nix

+17 −7
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@ 
  lib,

  stdenv,

  fetchFromGitHub,

  fetchpatch2,

  pkg-config,

  nss,

  efivar,
@@ -11,17 +12,26 @@ 
  mandoc,

}:



stdenv.mkDerivation rec {

stdenv.mkDerivation (finalAttrs: {

  pname = "pesign";

  version = "116";



  src = fetchFromGitHub {

    owner = "rhboot";

    repo = "pesign";

    rev = version;

    tag = finalAttrs.version;

    hash = "sha256-cuOSD/ZHkilgguDFJviIZCG8kceRWw2JgssQuWN02Do=";

  };



  patches = [

    # fix build with gcc14

    # https://github.com/rhboot/pesign/pull/119

    (fetchpatch2 {

      url = "https://github.com/rhboot/pesign/commit/1f9e2fa0b4d872fdd01ca3ba81b04dfb1211a187.patch?full_index=1";

      hash = "sha256-viVM4Z0jAEAWC3EdJVHcWe21aQskH5XE85lOd6Xd/qU=";

    })

  ];



  # nss-util is missing because it is already contained in nss

  # Red Hat seems to be shipping a separate nss-util:

  # https://centos.pkgs.org/7/centos-x86_64/nss-util-devel-3.44.0-4.el7_7.x86_64.rpm.html
@@ -49,12 +59,12 @@ stdenv.mkDerivation rec { 
    rm -rf $out/run

  '';



  meta = with lib; {

  meta = {

    description = "Signing tools for PE-COFF binaries. Compliant with the PE and Authenticode specifications";

    homepage = "https://github.com/rhboot/pesign";

    license = licenses.gpl2Only;

    maintainers = with maintainers; [ raitobezarius ];

    license = lib.licenses.gpl2Only;

    maintainers = with lib.maintainers; [ raitobezarius ];

    # efivar is currently Linux-only.

    platforms = platforms.linux;

    platforms = lib.platforms.linux;

  };

}
 
})