From: Maxim Cournoyer <maxim.cournoyer@gmail.com>
To: "Artyom V. Poptsov" <poptsov.artyom@gmail.com>
Cc: 71539-done@debbugs.gnu.org
Subject: bug#71539: [PATCH] gnu: astyle: Update to 3.5.
Date: Sun, 23 Jun 2024 22:18:51 -0400 [thread overview]
Message-ID: <87frt384ms.fsf@gmail.com> (raw)
In-Reply-To: <bb8cefa0c6300a0590b01a129bba1b4138436c8b.1718304845.git.poptsov.artyom@gmail.com> (Artyom V. Poptsov's message of "Thu, 13 Jun 2024 21:54:05 +0300")
Hi Artyom,
"Artyom V. Poptsov" <poptsov.artyom@gmail.com> writes:
> * gnu/packages/code.scm (astyle): Update to 3.5.
> [source]: Change source archive file name to match the upstream.
> [arguments]: Remove "modules". Add "patch-makefile" phase to set
> the C++ compiler to version c++17 as it is required for the build.
> Simplify "install-more" file.
Nitpick, there is no hanging indent is for GNU ChangeLogs.
> Change-Id: I8777710d723d22320c288c331645f8d61d5e640e
> ---
> gnu/packages/code.scm | 28 ++++++++++------------------
> 1 file changed, 10 insertions(+), 18 deletions(-)
>
> diff --git a/gnu/packages/code.scm b/gnu/packages/code.scm
> index d6ac8178f3..44edf43db5 100644
> --- a/gnu/packages/code.scm
> +++ b/gnu/packages/code.scm
> @@ -20,6 +20,7 @@
> ;;; Copyright © 2023 Fries <fries1234@protonmail.com>
> ;;; Copyright © 2023 Zheng Junjie <873216071@qq.com>
> ;;; Copyright © 2024 Sharlatan Hellseher <sharlatanus@gmail.com>
> +;;; Copyright © 2024 Artyom V. Poptsov <poptsov.artyom@gmail.com>
> ;;;
> ;;; This file is part of GNU Guix.
> ;;;
> @@ -823,14 +824,14 @@ (define-public uncrustify
> (define-public astyle
> (package
> (name "astyle")
> - (version "3.4.8")
> + (version "3.5")
> (source
> (origin
> (method url-fetch)
> (uri (string-append "mirror://sourceforge/astyle/astyle/astyle%20"
> - version "/astyle_" version "_linux.tar.gz"))
> + version "/astyle-" version ".tar.bz2"))
> (sha256
> - (base32 "1ms54wcs7hg1bsywqwf2lhdfizgbk7qxc9ghasxk8i99jvwlrk6b"))))
> + (base32 "0g4jyp47iz97ld9ac4wb5k59j4cs8dbw4dp8f32bwqx8pyvirz6y"))))
> (build-system gnu-build-system)
> (arguments
> (list
> @@ -839,11 +840,13 @@ (define-public astyle
> #~(list (string-append "prefix=" #$output)
> "INSTALL=install"
> "release" "shared")
> - #:modules '((guix build gnu-build-system) ;FIXME use %default-modules
> - (guix build utils)
> - (ice-9 regex))
> #:phases
> #~(modify-phases %standard-phases
> + (add-after 'unpack 'patch-makefile
> + (lambda _
> + (substitute* "build/gcc/Makefile"
> + (("CBASEFLAGS = -Wall -Wextra -fno-rtti -fno-exceptions -std=c\\+\\+11")
> + "CBASEFLAGS = -Wall -Wextra -fno-rtti -fno-exceptions -std=c++17"))))
That's breaking our max columns width of 80. I've broken the long
strings locally.
> (replace 'configure
> (lambda _
> (chdir "build/gcc")))
> @@ -852,12 +855,6 @@ (define-public astyle
> ;; Libraries and headers aren't installed by default.
> (let ((include (string-append #$output "/include"))
> (lib (string-append #$output "/lib")))
> - (define (link.so file strip-pattern)
> - (symlink
> - (basename file)
> - (regexp-substitute #f
> - (string-match strip-pattern file)
> - 'pre)))
> (mkdir-p include)
> (copy-file "../../src/astyle.h"
> (string-append include "/astyle.h"))
> @@ -865,12 +862,7 @@ (define-public astyle
> (for-each (lambda (l)
> (copy-file
> l (string-append lib "/" (basename l))))
> - (find-files "bin" "^lib.*\\.so"))
> - (for-each
> - (lambda (file)
> - (link.so file "(\\.[0-9]+){3}$") ;.so
> - (link.so file "(\\.[0-9]+){2}$")) ;.so.3
> - (find-files lib "lib.*\\.so\\..*"))))))))
> + (find-files "bin" "^lib.*\\.so"))))))))
> (home-page "https://astyle.sourceforge.net/")
> (synopsis "Source code indenter, formatter, and beautifier")
> (description
The rest LGTM, assuming their build system now takes care of what we
used to be doing manually.
Applied locally, will push shortly.
--
Thanks,
Maxim
prev parent reply other threads:[~2024-06-24 2:21 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-13 18:54 [bug#71539] [PATCH] gnu: astyle: Update to 3.5 Artyom V. Poptsov
2024-06-24 2:18 ` Maxim Cournoyer [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87frt384ms.fsf@gmail.com \
--to=maxim.cournoyer@gmail.com \
--cc=71539-done@debbugs.gnu.org \
--cc=poptsov.artyom@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).