all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: David Craven <david@craven.ch>
To: Danny Milosavljevic <dannym@scratchpost.org>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH v2 1/2] gnu: Add dtc.
Date: Thu, 1 Sep 2016 17:16:19 +0200	[thread overview]
Message-ID: <CAL1_imkNA_At+OpUqzaxUt5F+L6MFznUfmu+Tr_d9PNZo8aX5A@mail.gmail.com> (raw)
In-Reply-To: <20160831213437.31115-2-dannym@scratchpost.org>

So I'm ready to push dtc and u-boot. I'm still not sure about the
patches. If you want me to add them I think you should send a new
patch.

I see you dropped the uboot prefix from the patch names. The patch
names also need to be updated in local.mk.

I think that the 01 23 and 24 should be dropped, probably the libfdt
too. The patch names we have so far stick to lowercase patch names, so
we should too. And use - everywhere.

So that would be dtc-build-doc.patch,
dtc-add-missing-function-to-shared-library.patch and
dtc-add-some-missing-symbols-to-lds.patch.

If you make the changes I'll push your patch.

Thank you for your contribution!

David

  reply	other threads:[~2016-09-01 15:16 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-31 21:34 [PATCH v2 0/2] Add u-boot package and its dependent packages Danny Milosavljevic
2016-08-31 21:34 ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
2016-09-01 15:16   ` David Craven [this message]
2016-09-01 18:24     ` [PATCH] gnu: dtc: Add missing exports Danny Milosavljevic
2016-09-01 18:29     ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
2016-09-01 22:17       ` David Craven
2016-09-03 18:45       ` Danny Milosavljevic
2016-09-03 18:46         ` David Craven
2016-08-31 21:34 ` [PATCH v2 2/2] gnu: Add u-boot Danny Milosavljevic

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAL1_imkNA_At+OpUqzaxUt5F+L6MFznUfmu+Tr_d9PNZo8aX5A@mail.gmail.com \
    --to=david@craven.ch \
    --cc=dannym@scratchpost.org \
    --cc=guix-devel@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 external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.