From: Leo Famulari <leo@famulari.name>
To: Jan Nieuwenhuizen <janneke@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Add tcllib.
Date: Fri, 12 Feb 2016 18:05:30 -0500 [thread overview]
Message-ID: <20160212230530.GA28826@jasmine> (raw)
In-Reply-To: <20160212222322.GA27022@jasmine>
On Fri, Feb 12, 2016 at 05:23:22PM -0500, Leo Famulari wrote:
> On Fri, Feb 12, 2016 at 05:33:29PM +0100, Jan Nieuwenhuizen wrote:
> > Leo Famulari writes:
> >
> > >> Can you check if it works for you? I don't really know how to test this
> > >> package.
> > >
> > > My apologies is that patch series does not apply. I have attached
> > > another.
> >
> > Works for me; however, 0002-gnu-tcl-Use-tcl-tk-license.patch somehow
> > includes a bit from my tclxml patch (to be resent as soon as this gets
> > in)
> >
> > + gnu/packages/patches/tclxml-3.2-install.patch
>
> So I guess I actually *did* stay up too late ;)
>
> >
> > Also, hunk #1 of 0003 failed to apply for me.
>
> This kept happening to me! It's one reason I just made a new patch
> series from scratch, after manually addressing this failed hunk (your
> copyright line, how strange that it would not apply).
Okay, as discussed on IRC, I've pushed an updated patch series for you,
starting at d29f64707.
Thanks your contribution!
next prev parent reply other threads:[~2016-02-12 23:05 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-09 20:19 [PATCH] gnu: Add tcllib Jan Nieuwenhuizen
2016-02-11 19:53 ` Efraim Flashner
2016-02-11 20:28 ` Pjotr Prins
2016-02-11 20:42 ` Efraim Flashner
2016-02-11 21:40 ` Jan Nieuwenhuizen
2016-02-12 8:52 ` Ludovic Courtès
2016-02-11 21:11 ` Jan Nieuwenhuizen
2016-02-12 5:19 ` Leo Famulari
2016-02-12 5:29 ` Leo Famulari
2016-02-12 16:33 ` Jan Nieuwenhuizen
2016-02-12 22:23 ` Leo Famulari
2016-02-12 23:05 ` Leo Famulari [this message]
2016-02-12 8:47 ` Ludovic Courtès
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=20160212230530.GA28826@jasmine \
--to=leo@famulari.name \
--cc=guix-devel@gnu.org \
--cc=janneke@gnu.org \
/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).