Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Anders Darander <anders@chargestorm.se>
To: <openembedded-devel@lists.openembedded.org>
Subject: Re: [meta-oe][PATCH v3 2/2] ckermit: update to version 9.0
Date: Wed, 17 Aug 2011 10:26:28 +0200	[thread overview]
Message-ID: <20110817082628.GG7904@chargestorm.se> (raw)
In-Reply-To: <1313568501.4294.4.camel@mattotaupa>

* Paul Menzel <paulepanter@users.sourceforge.net> [110817 10:08]:
> Am Mittwoch, den 17.08.2011, 08:25 +0200 schrieb Anders Darander:
> > -LICENSE = "Kermit"
> > -LIC_FILES_CHKSUM = "file://COPYING.TXT;md5=17dabbd2543933464b6d32083c3a3596"
> > -SRC_URI = "ftp://kermit.columbia.edu/kermit/archives/cku${PV}.tar.gz;subdir=${BPN}-${PV}"
> > +LICENSE = "BSD-3-Clause"
> > +LIC_FILES_CHKSUM = "file://COPYING.TXT;md5=932ca542d6c6cb8a59a0bcd76ab67cc3"
> > +SRC_URI = "ftp://www.columbia.edu/kermit/archives/cku${PV}.tar.gz;subdir=${BPN}-${PV}"

> Koen already applied your patches, so this is just a note to please also
> add such changes to the commit message next time. Especially since the
> former URL seems to be the one used on the download page [1].

My mistake. When I started updating the recipe, the original url was not
working, and when submitting the patch series, I thought I had already
reverted that change. The non-working URL was probably only a temporary
glitch. Although I normally tend to review the patches one extra time,
to make sure that the commit message includes all changes, I missed this
one...

I've just submitted a new patch, reverting to the original SRC_URI, as
it _is_ working.

Thanks for spotting this!

Cheers,
Anders

-- 
Anders Darander
ChargeStorm AB	



      reply	other threads:[~2011-08-17  8:30 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-17  6:25 [meta-oe][PATCH v3 0/2] Import ckermit recipe Anders Darander
2011-08-17  6:25 ` [meta-oe][PATCH v3 1/2] ckermit: import from OE rev 5350bbef01b8979617e3d0fd38939f94f1fa5315 Anders Darander
2011-08-17  6:25 ` [meta-oe][PATCH v3 2/2] ckermit: update to version 9.0 Anders Darander
2011-08-17  8:08   ` Paul Menzel
2011-08-17  8:26     ` Anders Darander [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

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

  git send-email \
    --in-reply-to=20110817082628.GG7904@chargestorm.se \
    --to=anders@chargestorm.se \
    --cc=openembedded-devel@lists.openembedded.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox