Package: guix-patches;
Reported by: Léo Le Bouter <lle-bout <at> zaclys.net>
Date: Mon, 25 Jan 2021 10:23:01 UTC
Severity: normal
Tags: patch
Done: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Bug is archived. No further changes may be made.
View this message in rfc822 format
From: help-debbugs <at> gnu.org (GNU bug Tracking System) To: Léo Le Bouter <lle-bout <at> zaclys.net> Subject: bug#46095: closed (Re: bug#46095: [PATCH] gnu: Add mingw-w64-tools.) Date: Wed, 03 Feb 2021 15:30:02 +0000
[Message part 1 (text/plain, inline)]
Your bug report #46095: [PATCH] gnu: Add mingw-w64-tools. which was filed against the guix-patches package, has been closed. The explanation is attached below, along with your original report. If you require more details, please reply to 46095 <at> debbugs.gnu.org. -- 46095: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=46095 GNU Bug Tracking System Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: Léo Le Bouter <lle-bout <at> zaclys.net> Cc: 46095-done <at> debbugs.gnu.org Subject: Re: bug#46095: [PATCH] gnu: Add mingw-w64-tools. Date: Wed, 03 Feb 2021 10:29:22 -0500Hello, Maxim Cournoyer <maxim.cournoyer <at> gmail.com> writes: > Hello, > > Léo Le Bouter <lle-bout <at> zaclys.net> writes: > >> * gnu/packages/mingw.scm (mingw-w64-tools): New variable. >> --- >> gnu/packages/mingw.scm | 75 ++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 75 insertions(+) >> >> diff --git a/gnu/packages/mingw.scm b/gnu/packages/mingw.scm >> index b37f6c69bd..884cb7e39f 100644 >> --- a/gnu/packages/mingw.scm >> +++ b/gnu/packages/mingw.scm >> @@ -133,3 +133,78 @@ several new APIs such as DirectX and DDK, and 64-bit support.") >> #:with-winpthreads? #t)) >> >> (define-public mingw-w64 mingw-w64-i686) >> + >> +(define-public mingw-w64-tools >> + (package >> + (name "mingw-w64-tools") >> + (version "7.0.0") > > That doesn't seem to be the latest version. Is there a good reason why? > If so, please add a comment, else use the latest version. > >> + (source >> + (origin >> + (method url-fetch) >> + (uri (string-append >> + "mirror://sourceforge/mingw-w64/mingw-w64/" >> + "mingw-w64-release/mingw-w64-v" version ".tar.bz2")) >> + (sha256 >> + (base32 "0a5njsa2zw2ssdz10jkb10mhrf3cb8qp9avs89zqmw4n6pzxy85a")))) >> + (build-system gnu-build-system) >> + (arguments >> + `(#:modules (((guix build gnu-build-system) #:prefix gnu:) >> + ,@%gnu-build-system-modules) >> + #:phases >> + (modify-phases %standard-phases >> + (add-after 'unpack 'cd-gendef >> + (lambda _ (chdir "mingw-w64-tools/gendef") >> + #t)) > > It's a recent change, but ending phases by #t is no longer required. > >> + >> + (add-after 'install 'cd-genidl >> + (lambda _ (chdir "../genidl") >> + #t)) >> + (add-after 'cd-genidl 'configure-genidl >> + (assoc-ref gnu:%standard-phases 'configure)) >> + (add-after 'configure-genidl 'build-genidl >> + (assoc-ref gnu:%standard-phases 'build)) >> + (add-after 'build-genidl 'check-genidl >> + (assoc-ref gnu:%standard-phases 'check)) >> + (add-after 'check-genidl 'install-genidl >> + (assoc-ref gnu:%standard-phases 'install)) >> + >> + (add-after 'install-genidl 'cd-genlib >> + (lambda _ (chdir "../genlib") >> + #t)) >> + (add-after 'cd-genlib 'configure-genlib >> + (assoc-ref gnu:%standard-phases 'configure)) >> + (add-after 'configure-genlib 'build-genlib >> + (assoc-ref gnu:%standard-phases 'build)) >> + (add-after 'build-genlib 'check-genlib >> + (assoc-ref gnu:%standard-phases 'check)) >> + (add-after 'check-genlib 'install-genlib >> + (assoc-ref gnu:%standard-phases 'install)) >> + >> + (add-after 'install-genlib 'cd-genpeimg >> + (lambda _ (chdir "../genpeimg") >> + #t)) >> + (add-after 'cd-genpeimg 'configure-genpeimg >> + (assoc-ref gnu:%standard-phases 'configure)) >> + (add-after 'configure-genpeimg 'build-genpeimg >> + (assoc-ref gnu:%standard-phases 'build)) >> + (add-after 'build-genpeimg 'check-genpeimg >> + (assoc-ref gnu:%standard-phases 'check)) >> + (add-after 'check-genpeimg 'install-genpeimg >> + (assoc-ref gnu:%standard-phases 'install))))) >> + (home-page "https://mingw-w64.org") >> + (synopsis "Tools of Minimalist GNU for Windows") >> + (description >> + "This package provides tools of Minimalist GNU for Windows. >> + >> +Minimalist GNU for Windows (@dfn{MinGW}) is a complete software >> +development environment for creating native Microsoft Windows applications. >> + >> +It includes a set of Windows-specific header files and static import libraries >> +which enable the use of the Windows API. It does not rely on any third-party C >> +runtime dynamic-link libraries (@dfn{DLL}s). >> + >> +Mingw-w64 is an advancement of the original mingw.org project and provides >> +several new APIs such as DirectX and DDK, and 64-bit support.") > > I'd make it a single paragraph. > >> + (license (list >> + license:gpl3+ >> + license:lgpl2.1+)))) > > There should be a comment expliciting why there are two possible > licenses (different files? which ones? dual license? etc.) > > Thank you! > > Maxim I've addressed the above comments, refactored the phases and added the widl tool; you can see the result in commit 44c98c997933c4dc531775f1bb0848b39c7abf37. The version of mingw-w64 is now 8.0.0. Closing, Maxim
[Message part 3 (message/rfc822, inline)]
From: Léo Le Bouter <lle-bout <at> zaclys.net> To: guix-patches <at> gnu.org Cc: Léo Le Bouter <lle-bout <at> zaclys.net> Subject: [PATCH] gnu: Add mingw-w64-tools. Date: Mon, 25 Jan 2021 11:21:35 +0100* gnu/packages/mingw.scm (mingw-w64-tools): New variable. --- gnu/packages/mingw.scm | 75 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 75 insertions(+) diff --git a/gnu/packages/mingw.scm b/gnu/packages/mingw.scm index b37f6c69bd..884cb7e39f 100644 --- a/gnu/packages/mingw.scm +++ b/gnu/packages/mingw.scm @@ -133,3 +133,78 @@ several new APIs such as DirectX and DDK, and 64-bit support.") #:with-winpthreads? #t)) (define-public mingw-w64 mingw-w64-i686) + +(define-public mingw-w64-tools + (package + (name "mingw-w64-tools") + (version "7.0.0") + (source + (origin + (method url-fetch) + (uri (string-append + "mirror://sourceforge/mingw-w64/mingw-w64/" + "mingw-w64-release/mingw-w64-v" version ".tar.bz2")) + (sha256 + (base32 "0a5njsa2zw2ssdz10jkb10mhrf3cb8qp9avs89zqmw4n6pzxy85a")))) + (build-system gnu-build-system) + (arguments + `(#:modules (((guix build gnu-build-system) #:prefix gnu:) + ,@%gnu-build-system-modules) + #:phases + (modify-phases %standard-phases + (add-after 'unpack 'cd-gendef + (lambda _ (chdir "mingw-w64-tools/gendef") + #t)) + + (add-after 'install 'cd-genidl + (lambda _ (chdir "../genidl") + #t)) + (add-after 'cd-genidl 'configure-genidl + (assoc-ref gnu:%standard-phases 'configure)) + (add-after 'configure-genidl 'build-genidl + (assoc-ref gnu:%standard-phases 'build)) + (add-after 'build-genidl 'check-genidl + (assoc-ref gnu:%standard-phases 'check)) + (add-after 'check-genidl 'install-genidl + (assoc-ref gnu:%standard-phases 'install)) + + (add-after 'install-genidl 'cd-genlib + (lambda _ (chdir "../genlib") + #t)) + (add-after 'cd-genlib 'configure-genlib + (assoc-ref gnu:%standard-phases 'configure)) + (add-after 'configure-genlib 'build-genlib + (assoc-ref gnu:%standard-phases 'build)) + (add-after 'build-genlib 'check-genlib + (assoc-ref gnu:%standard-phases 'check)) + (add-after 'check-genlib 'install-genlib + (assoc-ref gnu:%standard-phases 'install)) + + (add-after 'install-genlib 'cd-genpeimg + (lambda _ (chdir "../genpeimg") + #t)) + (add-after 'cd-genpeimg 'configure-genpeimg + (assoc-ref gnu:%standard-phases 'configure)) + (add-after 'configure-genpeimg 'build-genpeimg + (assoc-ref gnu:%standard-phases 'build)) + (add-after 'build-genpeimg 'check-genpeimg + (assoc-ref gnu:%standard-phases 'check)) + (add-after 'check-genpeimg 'install-genpeimg + (assoc-ref gnu:%standard-phases 'install))))) + (home-page "https://mingw-w64.org") + (synopsis "Tools of Minimalist GNU for Windows") + (description + "This package provides tools of Minimalist GNU for Windows. + +Minimalist GNU for Windows (@dfn{MinGW}) is a complete software +development environment for creating native Microsoft Windows applications. + +It includes a set of Windows-specific header files and static import libraries +which enable the use of the Windows API. It does not rely on any third-party C +runtime dynamic-link libraries (@dfn{DLL}s). + +Mingw-w64 is an advancement of the original mingw.org project and provides +several new APIs such as DirectX and DDK, and 64-bit support.") + (license (list + license:gpl3+ + license:lgpl2.1+)))) -- 2.30.0
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.