From: Aaron Covrig via Guix-patches via <guix-patches@gnu.org>
To: John Kehayias <john.kehayias@protonmail.com>
Cc: 65330@debbugs.gnu.org
Subject: [bug#65330] [PATCH] Fixing tilix compile issue by upgrading version
Date: Tue, 22 Aug 2023 21:45:54 -0400 [thread overview]
Message-ID: <20230822214554.28cd7227@mobile-dev> (raw)
In-Reply-To: <87y1i2j3oa.fsf_-_@protonmail.com>
[-- Attachment #1: Type: text/plain, Size: 3269 bytes --]
Awesome, thank you John for your assistance and reviewing!
v/r,
Aaron Covrig
On Tue, 22 Aug 2023 19:08:26 +0000
John Kehayias <john.kehayias@protonmail.com> wrote:
> Hello,
>
> On Mon, Aug 21, 2023 at 12:26 AM, Aaron Covrig wrote:
>
> > 1. Changed tilix from v1.9.5 to v1.9.5 + commits
> > ---
> > This revision changes the version to reflect that this is built
> > upon v1.9.5 with added commits, rather than being v1.9.6;
> > additionally, it corrects line spacing so that only the
> > changed parts are seen as modified by git
>
> We were discussing on IRC the other day. I meant that no changes to
> the formatting besides the spacing introduced by the let should be
> made; this still requires re-indenting the rest of the package
> definition.
>
> Anyway, I adjusted the commit message, fixed the formatting, and used
> the latest commit of tilix. I checked that this built locally as well.
>
> Thanks for your contribution!
>
> John
>
> > gnu/packages/terminals.scm | 26 +++++++++++++++-----------
> > 1 file changed, 15 insertions(+), 11 deletions(-)
> >
> > diff --git a/gnu/packages/terminals.scm b/gnu/packages/terminals.scm
> > index 5fc72d7937..fd55233a5b 100644
> > --- a/gnu/packages/terminals.scm
> > +++ b/gnu/packages/terminals.scm
> > @@ -33,6 +33,7 @@
> > ;;; Copyright © 2022 Felipe Balbi <balbi@kernel.org>
> > ;;; Copyright © 2022 ( <paren@disroot.org>
> > ;;; Copyright © 2022 jgart <jgart@dismail.de>
> > +;;; Copyright © 2023 Aaron Covrig <aaron.covrig.us@ieee.org>
> > ;;;
> > ;;; This file is part of GNU Guix.
> > ;;;
> > @@ -1339,18 +1340,21 @@ (define-public wterm
> > (deprecated-package "wterm" foot))
> >
> > (define-public tilix
> > +(let ((commit "e2c75f417a84ad784735524f1b8bf01c516f705d")
> > + (revision "0"))
> > (package
> > (name "tilix")
> > - (version "1.9.5")
> > - (source
> > - (origin
> > - (method git-fetch)
> > - (uri (git-reference
> > - (url "https://github.com/gnunn1/tilix")
> > - (commit version)))
> > - (file-name (git-file-name name version))
> > - (sha256
> > - (base32
> > "1ij3ix6yhi8hicxvglrxjyyv8bch9birrgsr8ml6jfh3hvk4pxdh"))))
> > + ;; includes commits that fix glibc related build issues
> > + (version (git-version "1.9.5" revision commit))
> > + (source (origin
> > + (method git-fetch)
> > + (uri (git-reference
> > + (url "https://github.com/gnunn1/tilix")
> > + (commit commit)))
> > + (file-name (git-file-name name version))
> > + (sha256
> > + (base32
> > +
> > "1k5r99c9ab1l0v37hs1zy08rspgv8n7y3bd13ljqg607cyalbgbf"))))
> > (build-system meson-build-system) (arguments
> > `(#:glib-or-gtk? #t
> > @@ -1394,7 +1398,7 @@ (define-public tilix
> > terminal are replicated to the others.
> > @item Supports notifications when processes are completed out of
> > view. @end enumerate")
> > - (license license:mpl2.0)))
> > + (license license:mpl2.0))))
> >
> > (define-public tio
> > (package
> >
> > base-commit: 7cad70566322262f6acdbbb965e867e34010eb55
>
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
prev parent reply other threads:[~2023-08-23 1:47 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-16 1:34 [bug#65330] [PATCH] Fixing tilix compile issue by upgrading version Aaron Covrig via Guix-patches via
2023-08-21 4:26 ` [bug#65330] [PATCH v1] Fixing tilix compile issues by including updates Aaron Covrig via Guix-patches via
2023-08-22 19:08 ` bug#65330: [PATCH] Fixing tilix compile issue by upgrading version John Kehayias via Guix-patches via
2023-08-23 1:45 ` Aaron Covrig via Guix-patches via [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=20230822214554.28cd7227@mobile-dev \
--to=guix-patches@gnu.org \
--cc=65330@debbugs.gnu.org \
--cc=aaron.covrig.us@ieee.org \
--cc=john.kehayias@protonmail.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).