| Submitter | Steffen Sledz |
|---|---|
| Date | March 21, 2012, 10:37 a.m. |
| Message ID | <1332326252-7381-1-git-send-email-sledz@dresearch-fe.de> |
| Download | mbox | patch |
| Permalink | /patch/23961/ |
| State | Accepted |
| Commit | d04bee54dc1b54faf247d47ce2ff88a6660f76e9 |
| Headers | show |
Comments
2012/3/21 Steffen Sledz <sledz@dresearch-fe.de>: > libmatthew-java-0.7.1.tar.gz is no longer available at the original > location. > > Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> > --- > recipes/libmatthew/libmatthew_0.7.1.bb | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/recipes/libmatthew/libmatthew_0.7.1.bb b/recipes/libmatthew/libmatthew_0.7.1.bb > index 266a0d4..6916e56 100644 > --- a/recipes/libmatthew/libmatthew_0.7.1.bb > +++ b/recipes/libmatthew/libmatthew_0.7.1.bb > @@ -2,7 +2,7 @@ require libmatthew.inc > > PR = "r0" > > -SRC_URI = "http://www.matthew.ath.cx/projects/java/libmatthew-java-${PV}.tar.gz \ > +SRC_URI = "http://pkgs.fedoraproject.org/repo/pkgs/libmatthew-java/libmatthew-java-0.7.1.tar.gz/6a4db221129f230c64a0f937d00bb703/libmatthew-java-${PV}.tar.gz \ > file://Makefile-0.7.patch" > > > -- > 1.7.7 > Acked-by: Frans Meulenbroeks <fransmeulenbroeks@gmail.com>
On Wed, Mar 21, 2012 at 3:37 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote: > libmatthew-java-0.7.1.tar.gz is no longer available at the original > location. > > Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> NAK, please switch it to the snapshot.debian.org URL I provided so we don't have to change this recipe at some point in the future again.
On Wed, Mar 21, 2012 at 7:44 AM, Tom Rini <tom.rini@gmail.com> wrote: > On Wed, Mar 21, 2012 at 3:37 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote: >> libmatthew-java-0.7.1.tar.gz is no longer available at the original >> location. >> >> Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> > > NAK, please switch it to the snapshot.debian.org URL I provided so we > don't have to change this recipe at some point in the future again. On third thought, lemme think about this. I don't like pointing somewhere that might remove it, again, but I also don't like changing checksums for folks that already have it down. So, unless someone else disagrees, we'll favor same checksums over retention and if fedoraproject removes it, we'll just point at the copy on the OE server.
On 21.03.2012 15:44, Tom Rini wrote: > On Wed, Mar 21, 2012 at 3:37 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote: >> libmatthew-java-0.7.1.tar.gz is no longer available at the original >> location. >> >> Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> > > NAK, please switch it to the snapshot.debian.org URL I provided so we > don't have to change this recipe at some point in the future again. As i mentioned in another subthread the URL you provided points to tarball which differs from the one referenced in the recipe. :(
On 21.03.2012 15:46, Tom Rini wrote: > On Wed, Mar 21, 2012 at 7:44 AM, Tom Rini <tom.rini@gmail.com> wrote: >> On Wed, Mar 21, 2012 at 3:37 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote: >>> libmatthew-java-0.7.1.tar.gz is no longer available at the original >>> location. >>> >>> Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> >> >> NAK, please switch it to the snapshot.debian.org URL I provided so we >> don't have to change this recipe at some point in the future again. > > On third thought, lemme think about this. I don't like pointing > somewhere that might remove it, again, but I also don't like changing > checksums for folks that already have it down. So, unless someone > else disagrees, we'll favor same checksums over retention and if > fedoraproject removes it, we'll just point at the copy on the OE > server. So finally that's an ACK for my patch?
On Thu, Mar 22, 2012 at 08:01:41AM +0100, Steffen Sledz wrote: > On 21.03.2012 15:46, Tom Rini wrote: > > On Wed, Mar 21, 2012 at 7:44 AM, Tom Rini <tom.rini@gmail.com> wrote: > >> On Wed, Mar 21, 2012 at 3:37 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote: > >>> libmatthew-java-0.7.1.tar.gz is no longer available at the original > >>> location. > >>> > >>> Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> > >> > >> NAK, please switch it to the snapshot.debian.org URL I provided so we > >> don't have to change this recipe at some point in the future again. > > > > On third thought, lemme think about this. I don't like pointing > > somewhere that might remove it, again, but I also don't like changing > > checksums for folks that already have it down. So, unless someone > > else disagrees, we'll favor same checksums over retention and if > > fedoraproject removes it, we'll just point at the copy on the OE > > server. > > So finally that's an ACK for my patch? Yes, thanks.
On 22.03.2012 23:13, Tom Rini wrote: > On Thu, Mar 22, 2012 at 08:01:41AM +0100, Steffen Sledz wrote: >> On 21.03.2012 15:46, Tom Rini wrote: >>> On Wed, Mar 21, 2012 at 7:44 AM, Tom Rini <tom.rini@gmail.com> wrote: >>>> On Wed, Mar 21, 2012 at 3:37 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote: >>>>> libmatthew-java-0.7.1.tar.gz is no longer available at the original >>>>> location. >>>>> >>>>> Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> >>>> >>>> NAK, please switch it to the snapshot.debian.org URL I provided so we >>>> don't have to change this recipe at some point in the future again. >>> >>> On third thought, lemme think about this. I don't like pointing >>> somewhere that might remove it, again, but I also don't like changing >>> checksums for folks that already have it down. So, unless someone >>> else disagrees, we'll favor same checksums over retention and if >>> fedoraproject removes it, we'll just point at the copy on the OE >>> server. >> >> So finally that's an ACK for my patch? > > Yes, thanks. OK, pushed to oe-classic master. Tom, do you need an extra pull request for 2011.03-maintenance branch?
On Fri, Mar 23, 2012 at 08:01:36AM +0100, Steffen Sledz wrote: > On 22.03.2012 23:13, Tom Rini wrote: > > On Thu, Mar 22, 2012 at 08:01:41AM +0100, Steffen Sledz wrote: > >> On 21.03.2012 15:46, Tom Rini wrote: > >>> On Wed, Mar 21, 2012 at 7:44 AM, Tom Rini <tom.rini@gmail.com> wrote: > >>>> On Wed, Mar 21, 2012 at 3:37 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote: > >>>>> libmatthew-java-0.7.1.tar.gz is no longer available at the original > >>>>> location. > >>>>> > >>>>> Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> > >>>> > >>>> NAK, please switch it to the snapshot.debian.org URL I provided so we > >>>> don't have to change this recipe at some point in the future again. > >>> > >>> On third thought, lemme think about this. I don't like pointing > >>> somewhere that might remove it, again, but I also don't like changing > >>> checksums for folks that already have it down. So, unless someone > >>> else disagrees, we'll favor same checksums over retention and if > >>> fedoraproject removes it, we'll just point at the copy on the OE > >>> server. > >> > >> So finally that's an ACK for my patch? > > > > Yes, thanks. > > OK, pushed to oe-classic master. > > Tom, do you need an extra pull request for 2011.03-maintenance branch? Yes please. Thanks!
Patch
diff --git a/recipes/libmatthew/libmatthew_0.7.1.bb b/recipes/libmatthew/libmatthew_0.7.1.bb index 266a0d4..6916e56 100644 --- a/recipes/libmatthew/libmatthew_0.7.1.bb +++ b/recipes/libmatthew/libmatthew_0.7.1.bb @@ -2,7 +2,7 @@ require libmatthew.inc PR = "r0" -SRC_URI = "http://www.matthew.ath.cx/projects/java/libmatthew-java-${PV}.tar.gz \ +SRC_URI = "http://pkgs.fedoraproject.org/repo/pkgs/libmatthew-java/libmatthew-java-0.7.1.tar.gz/6a4db221129f230c64a0f937d00bb703/libmatthew-java-${PV}.tar.gz \ file://Makefile-0.7.patch"
libmatthew-java-0.7.1.tar.gz is no longer available at the original location. Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de> --- recipes/libmatthew/libmatthew_0.7.1.bb | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)