diff mbox series

[11/14] package/tidsp-binaries: URL update

Message ID 1537449899-9576-11-git-send-email-matthew.weber@rockwellcollins.com
State Rejected
Headers show
Series [01/14] support/scripts/pkg-stats: URL checking support | expand

Commit Message

Matt Weber Sept. 20, 2018, 1:24 p.m. UTC
Signed-off-by: Matt Weber <matthew.weber@rockwellcollins.com>
---
 package/tidsp-binaries/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Petazzoni Sept. 20, 2018, 7:39 p.m. UTC | #1
Hello,

On Thu, 20 Sep 2018 08:24:56 -0500, Matt Weber wrote:
> Signed-off-by: Matt Weber <matthew.weber@rockwellcollins.com>
> ---
>  package/tidsp-binaries/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/tidsp-binaries/Config.in b/package/tidsp-binaries/Config.in
> index 48f85a5..24b2891 100644
> --- a/package/tidsp-binaries/Config.in
> +++ b/package/tidsp-binaries/Config.in
> @@ -4,4 +4,4 @@ config BR2_PACKAGE_TIDSP_BINARIES
>  	help
>  	  TI OMAP3 DSP algorithms.
>  
> -	  https://gforge.ti.com/gf/project/openmax/
> +	  http://omappedia.org/wiki/OpenMAX_Project

Is that really the right URL ? It points to gforge.ti.com, which indeed
seems to be down. Should we simply drop this package perhaps ? The
upstream site is not available, I doubt anybody is using it: it was
introduced in 2011 and never touched again in any useful/meaningful way.

So I would propose to drop this package.

Thomas
Matt Weber Sept. 20, 2018, 9:38 p.m. UTC | #2
Thomas,

On Thu, Sep 20, 2018 at 2:39 PM Thomas Petazzoni
<thomas.petazzoni@bootlin.com> wrote:
>
> Hello,
>
> On Thu, 20 Sep 2018 08:24:56 -0500, Matt Weber wrote:
> > Signed-off-by: Matt Weber <matthew.weber@rockwellcollins.com>
> > ---
> >  package/tidsp-binaries/Config.in | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/package/tidsp-binaries/Config.in b/package/tidsp-binaries/Config.in
> > index 48f85a5..24b2891 100644
> > --- a/package/tidsp-binaries/Config.in
> > +++ b/package/tidsp-binaries/Config.in
> > @@ -4,4 +4,4 @@ config BR2_PACKAGE_TIDSP_BINARIES
> >       help
> >         TI OMAP3 DSP algorithms.
> >
> > -       https://gforge.ti.com/gf/project/openmax/
> > +       http://omappedia.org/wiki/OpenMAX_Project
>
> Is that really the right URL ? It points to gforge.ti.com, which indeed
> seems to be down. Should we simply drop this package perhaps ? The
> upstream site is not available, I doubt anybody is using it: it was
> introduced in 2011 and never touched again in any useful/meaningful way.
>
> So I would propose to drop this package.

This package is used by dsp-tools and gstreamer/gst-dsp so I'd assume
those should also be removed.

This capability feels feature complete when looking at the Gstreamer
stuff....  I guess we'll see if anyone notices the removal.  Should I
have 3 seperate patches for the individual packages?

Matt
diff mbox series

Patch

diff --git a/package/tidsp-binaries/Config.in b/package/tidsp-binaries/Config.in
index 48f85a5..24b2891 100644
--- a/package/tidsp-binaries/Config.in
+++ b/package/tidsp-binaries/Config.in
@@ -4,4 +4,4 @@  config BR2_PACKAGE_TIDSP_BINARIES
 	help
 	  TI OMAP3 DSP algorithms.
 
-	  https://gforge.ti.com/gf/project/openmax/
+	  http://omappedia.org/wiki/OpenMAX_Project