search

samjabrahams / tensorflow-on-raspberry-pi

TensorFlow for Raspberry Pi
Other
2.25k stars 496 forks source link

protobuf autogen.sh file contains invalid links to gmock-1.7.0.zip #42

Open VALARMO opened 8 years ago

VALARMO commented 8 years ago

Hardware/Software Info

[gmock-1.7.0.zip] End-of-central-directory signature not found. Either this file is not a zipfile, or it constitutes one disk of a multi-part archive. In the latter case the central directory and zipfile comment will be found on the last disk(s) of this archive. unzip: cannot find zipfile directory in one of gmock-1.7.0.zip or gmock-1.7.0.zip.zip, and cannot find gmock-1.7.0.zip.ZIP, period.

fix

simple edit autogen.sh to change the link to download gmock-1.70 as this: http://pkgs.fedoraproject.org/repo/pkgs/gmock/gmock-1.7.0.zip/073b984d8798ea1594f5e44d85b20d66/gmock-1.7.0.zip

jalajc commented 8 years ago

I too faced the same issue, however I think the fix which keep pace with google mock is...

modifying the autogen.sh as per https://github.com/google/protobuf/pull/2260/files

-->starting next line

if test ! -e gmock -o   && ! -e googlemock-release-1.7.0; then
   echo "Google Mock not present.  Fetching gmock-1.7.0 from the web..."
   curl $curlopts -L -O https://github.com/google/googlemock/archive/release-1.7.0.zip
   unzip -q release-1.7.0.zip
   rm release-1.7.0.zip
   mv googlemock-release-1.7.0 gmock

   curl $curlopts -L -O https://github.com/google/googletest/archive/release-1.7.0.zip
   unzip -q release-1.7.0.zip
   rm release-1.7.0.zip
   mv googletest-release-1.7.0 gmock/gtest
 fi

-->end at fi

neuralearner commented 8 years ago

Alternatively, you can get it straight from fedora http://pkgs.fedoraproject.org/repo/pkgs/gmock/gmock-1.7.0.zip/073b984d8798ea1594f5e44d85b20d66/gmock-1.7.0.zip

I installed and it worked fine as well.

Edit: hmm, lead to a java compile error down the line, I'll give yours a try. Hmm, java issue seems to be unrelated to this.

jtoy commented 7 years ago

same issue as @neuralearner

samjabrahams commented 7 years ago

Thanks for the information @VALARMO @jalajc @neuralearner @jtoy - I need to do a fresh install of Rasbian and update the guide. I get back from traveling tonight, and I'll try to prioritize this repo over this week to get everything up to date and correct.

johnnogent commented 7 years ago

or you can use another branch V2.3.x