[Buildroot] [PATCH] eeprog: fix homepage link

[Buildroot] [PATCH] eeprog: fix homepage link

[Baruch Siach]

The current link leads to a 400 Bad Request error page.

Cc: Matt Weber <matthew.weber@rockwellcollins.com>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 package/eeprog/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/eeprog/Config.in b/package/eeprog/Config.in
index ff313bdbae96..73582c70f9d4 100644
--- a/package/eeprog/Config.in
+++ b/package/eeprog/Config.in
@@ -3,4 +3,4 @@ config BR2_PACKAGE_EEPROG
 	help
 	  Simple tool to read/write i2c eeprom chips.
 
-	  http://codesink.org/eeprog.html
+	  http://www.codesink.org/eeprog.html
-- 
2.15.1

[Buildroot] [PATCH] eeprog: fix homepage link

[Thomas Petazzoni]

Hello,

On Sun, 31 Dec 2017 13:25:06 +0200, Baruch Siach wrote:
> The current link leads to a 400 Bad Request error page.
> 
> Cc: Matt Weber <matthew.weber@rockwellcollins.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  package/eeprog/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to master, thanks. However, do we still need this package, now
that i2c-tools also builds/installs eeprog ?

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] eeprog: fix homepage link

[Baruch Siach]

Hi Thomas,

On Sun, Dec 31, 2017 at 12:48:12PM +0100, Thomas Petazzoni wrote:
> On Sun, 31 Dec 2017 13:25:06 +0200, Baruch Siach wrote:
> > The current link leads to a 400 Bad Request error page.
> > 
> > Cc: Matt Weber <matthew.weber@rockwellcollins.com>
> > Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> > ---
> >  package/eeprog/Config.in | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Applied to master, thanks. However, do we still need this package, now
> that i2c-tools also builds/installs eeprog ?

Probably not. Event worse. When both i2c-tools and eeprog are enabled, the 
target /usr/bin/eeprog depends on the build order. Has the autobuilder caught 
this file overwrite?

Are you OK with dropping eeprog? Matt?

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

[Buildroot] [PATCH] eeprog: fix homepage link

[Thomas Petazzoni]

Hello,

On Sun, 31 Dec 2017 14:26:18 +0200, Baruch Siach wrote:

> > Applied to master, thanks. However, do we still need this package, now
> > that i2c-tools also builds/installs eeprog ?  
> 
> Probably not.

Do they provide the same features ?

> Event worse. When both i2c-tools and eeprog are enabled, the 
> target /usr/bin/eeprog depends on the build order. Has the autobuilder caught 
> this file overwrite?

It will, be currently such errors are not errors, but warnings: they
don't make the build fail. Yann recently posted a patch that would
allow to turn such warnings into errors, but there are so many of such
overwrites today that enabling such an option in the autobuilders would
turn almost all results to NOK.

> Are you OK with dropping eeprog? Matt?

I don't use eeprog, so I don't really care. But if the standalone
eeprog and the one on i2c-tools are equivalent, I think it makes sense
to keep the one from i2c-tools.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[Buildroot] [PATCH] eeprog: fix homepage link

[Baruch Siach]

Hi Thomas,

On Sun, Dec 31, 2017 at 01:54:33PM +0100, Thomas Petazzoni wrote:
> On Sun, 31 Dec 2017 14:26:18 +0200, Baruch Siach wrote:
> > > Applied to master, thanks. However, do we still need this package, now
> > > that i2c-tools also builds/installs eeprog ?  
> > 
> > Probably not.
> 
> Do they provide the same features ?

They are both based on the same code. It looks like i2c-tools adopted eeprog 
after the last eeprog release in February 2004.

> > Event worse. When both i2c-tools and eeprog are enabled, the target 
> > /usr/bin/eeprog depends on the build order. Has the autobuilder caught 
> > this file overwrite?
> 
> It will, be currently such errors are not errors, but warnings: they
> don't make the build fail. Yann recently posted a patch that would
> allow to turn such warnings into errors, but there are so many of such
> overwrites today that enabling such an option in the autobuilders would
> turn almost all results to NOK.
> 
> > Are you OK with dropping eeprog? Matt?
> 
> I don't use eeprog, so I don't really care. But if the standalone
> eeprog and the one on i2c-tools are equivalent, I think it makes sense
> to keep the one from i2c-tools.

OK. Let's give Matt some time to recover from the holidays and respond before 
I send a patch to remove the eeprog package.

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

[Buildroot] [PATCH] eeprog: fix homepage link

[Thomas Petazzoni]

Hello,

On Sun, 31 Dec 2017 15:11:08 +0200, Baruch Siach wrote:

> They are both based on the same code. It looks like i2c-tools adopted eeprog 
> after the last eeprog release in February 2004.

OK.

> OK. Let's give Matt some time to recover from the holidays and respond before 
> I send a patch to remove the eeprog package.

Sounds good. Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[Buildroot] [PATCH] eeprog: fix homepage link

[Matthew Weber]

Baruch, Thomas,

On Sun, Dec 31, 2017 at 10:08 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
>> OK. Let's give Matt some time to recover from the holidays and respond before
>> I send a patch to remove the eeprog package.
>

Removal sounds good to me.

Angelo, thanks for bumping i2c-tools!  Lots of good stuff with this
cleanup and the new tools we now install.

Matt

[Buildroot] [PATCH] eeprog: fix homepage link

[Peter Korsgaard]

>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:

 > The current link leads to a 400 Bad Request error page.
 > Cc: Matt Weber <matthew.weber@rockwellcollins.com>
 > Signed-off-by: Baruch Siach <baruch@tkos.co.il>

Committed to 2017.11.x, thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH] eeprog: fix homepage link

[Peter Korsgaard]

>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:

 > The current link leads to a 400 Bad Request error page.
 > Cc: Matt Weber <matthew.weber@rockwellcollins.com>
 > Signed-off-by: Baruch Siach <baruch@tkos.co.il>

Committed to 2017.02.x, thanks.

-- 
Bye, Peter Korsgaard