Skip to contentSkip to conversation

fix cjson package

PATCH New Viallard Anthony 3 participants 5 messages Dec 6, 2012 – Dec 10, 2012

lore
Share & tools
lore

Viallard Anthony

author
#0 lore raw

PATCH 1/1 fix cjson package

Hi buildroot guys,

I've try to use cjson package but there a download problem. So, I fix up the package Makefile and here the change :

package/cjson/cjson.mk +2 -2
index ccc6010..728639d 100644
@@ -3,8 +3,8 @@
  # cjson
  #
  #############################################################
-CJSON_VERSION         = undefined
-CJSON_SOURCE          = cJSONFiles.zip
+CJSON_VERSION         = 2011-10-10
+CJSON_SOURCE          = cJSONFiles-$(CJSON_VERSION).zip
  CJSON_SITE            = http://downloads.sourceforge.net/project/cjson/
  CJSON_INSTALL_STAGING = YES
  CJSON_LICENSE         = MIT

Cheers,
Viallard Anthony.

#1 lore raw

PATCH 1/1 fix cjson package

Viallard,

package/cjson/cjson.mk line 8

Quoted 5 lines
package/cjson/cjson.mk +2 -2
-CJSON_VERSION         = undefined
-CJSON_SOURCE          = cJSONFiles.zip
+CJSON_VERSION         = 2011-10-10
+CJSON_SOURCE          = cJSONFiles-$(CJSON_VERSION).zip
 CJSON_SITE            = http://downloads.sourceforge.net/project/cjson/

This worked for you? It doesn't work for me ... i.e. this line works for me:

wget http://downloads.sourceforge.net/project/cjson/cJSONFiles.zip

but this line gives me ERROR 404 Not Found:

    wget http://downloads.sourceforge.net/project/cjson/cJSONFiles-2011-10-10.zip

Am I missing something here?

Danomi -

author
#2 lore raw

PATCH 1/1 fix cjson package

Quoted 11 lines · Danomi Manchego

Viallard,

Review

Quoted 5 lines
(quoted diff) +2 -2
-CJSON_VERSION         = undefined
-CJSON_SOURCE          = cJSONFiles.zip
+CJSON_VERSION         = 2011-10-10
+CJSON_SOURCE          = cJSONFiles-$(CJSON_VERSION).zip
  CJSON_SITE            = http://downloads.sourceforge.net/project/cjson/

This worked for you? It doesn't work for me ... i.e. this line works for me:

wget http://downloads.sourceforge.net/project/cjson/cJSONFiles.zip

Quoted 7 lines

but this line gives me ERROR 404 Not Found:

     wget http://downloads.sourceforge.net/project/cjson/cJSONFiles-2011-10-10.zip

Am I missing something here?

Danomi -

Actually, "wget
http://downloads.sourceforge.net/project/cjson/cJSONFiles.zip" works and the other link not indeed. I just realize I've a "Backup download site" with cJSONFiles-2011-10-10.zip on it ...

But there really a problem when buildroot trying to download the file :

-----------------------
avd at avd-box:~/wksp/buildroot$ make cjson-build

Quoted 1 line
Quoted 1 line
Quoted 1 line

cjson undefined Downloading

--2012-12-07 09:24:34-- http://192.168.2.17/files/buildroot/cJSONFiles.zip
Connexion vers 192.168.2.17:80...connect?.
requ?te HTTP transmise, en attente de la r?ponse...404 Not Found

2012-12-07 09:24:34 ERREUR 404: Not Found.

--2012-12-07 09:24:34--
http://downloads.sourceforge.net/project/cjson//cJSONFiles.zip
R?solution de downloads.sourceforge.net (downloads.sourceforge.net)... 216.34.181.59
Connexion vers downloads.sourceforge.net (downloads.sourceforge.net)|216.34.181.59|:80...connect?. requ?te HTTP transmise, en attente de la r?ponse...404 Not Found

2012-12-07 09:24:35 ERREUR 404: Not Found.

--2012-12-07 09:24:35-- http://192.168.2.17/files/buildroot/cJSONFiles.zip
Connexion vers 192.168.2.17:80...connect?.
requ?te HTTP transmise, en attente de la r?ponse...404 Not Found

2012-12-07 09:24:35 ERREUR 404: Not Found.

make: *** [/home/avd/wksp/buildroot/output/build/cjson-undefined/.stamp_downloaded] Erreur 1
-----------------------

And i know now what is the actual issue. Here the good big fix :

---
  package/cjson/cjson.mk |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
package/cjson/cjson.mk +1 -1
index ccc6010..55af800 100644
@@ -5,7 +5,7 @@
  #############################################################
  CJSON_VERSION         = undefined
  CJSON_SOURCE          = cJSONFiles.zip
-CJSON_SITE            = http://downloads.sourceforge.net/project/cjson/
+CJSON_SITE            = http://downloads.sourceforge.net/project/cjson
  CJSON_INSTALL_STAGING = YES
  CJSON_LICENSE         = MIT
Signature · 4 lines
1.7.10.4

Cheers,
avd.
#3 lore raw

PATCH 1/1 fix cjson package

Quoted 21 lines · Viallard Anthony

And i know now what is the actual issue. Here the good big fix :

---
 package/cjson/cjson.mk |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
package/cjson/cjson.mk +1 -2
index ccc6010..55af800 100644
@@ -5,7 +5,7 @@
 #############################################################
 CJSON_VERSION         = undefined
 CJSON_SOURCE          = cJSONFiles.zip
-CJSON_SITE            = http://downloads.sourceforge.net/project/cjson/
+CJSON_SITE            = http://downloads.sourceforge.net/project/cjson
 
 CJSON_INSTALL_STAGING = YES
 CJSON_LICENSE         = MIT
 
--
Quoted 1 line

1.7.10.4

I'm not sure if Peter needs you to send the good fix as a git-send-email'ed V2, but:

Acked-by Danomi Manchego <[email protected]>
#4 lore raw

PATCH 1/1 fix cjson package

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Danomi" == Danomi Manchego <[email protected]> writes:

Hi,

Danomi> I'm not sure if Peter needs you to send the good fix as a Danomi> git-send-email'ed V2, but:

Danomi> Acked-by: Danomi Manchego <[email protected]>

Yes, preferably - But I've created a git commit myself for this - Thanks.

Signature · 1 line
Bye, Peter Korsgaard