Patchwork libmatthew-0.7.1: SRC_URI updated

login
register
mail settings
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

Steffen Sledz - March 21, 2012, 10:37 a.m.
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(-)
Frans Meulenbroeks - March 21, 2012, 12:58 p.m.
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>
Tom Rini - March 21, 2012, 2:44 p.m.
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.
Tom Rini - March 21, 2012, 2:46 p.m.
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.
Steffen Sledz - March 21, 2012, 2:47 p.m.
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. :(
Steffen Sledz - March 22, 2012, 7:01 a.m.
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?
Tom Rini - March 22, 2012, 10:13 p.m.
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.
Steffen Sledz - March 23, 2012, 7:01 a.m.
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?
Tom Rini - March 23, 2012, 3:20 p.m.
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"