Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ppa for gflag and glog #1077

Merged
merged 3 commits into from
Sep 14, 2014
Merged

Add ppa for gflag and glog #1077

merged 3 commits into from
Sep 14, 2014

Conversation

bhack
Copy link
Contributor

@bhack bhack commented Sep 12, 2014

Speedup travis build

@kloudkl
Copy link
Contributor

kloudkl commented Sep 13, 2014

There are multiple lines of apt-get -y update. Will the building speed be faster by merging them into one?

@bhack
Copy link
Contributor Author

bhack commented Sep 13, 2014

Probably a little bit. But this will need to duplicate the if on cmake. One for adding repository and one for the cmake installation.

@bhack
Copy link
Contributor Author

bhack commented Sep 13, 2014

@jeffdonahue What do u think? Can be merged?

@@ -6,6 +6,7 @@ set -e
MAKE="make --jobs=$NUM_THREADS"

# Install apt packages where the Ubuntu 12.04 default works for Caffe
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please change this comment accordingly since these aren't the Ubuntu 12.04 default, and also add a comment above the add-apt-repository saying # for gflags and glog or something to that effect

@jeffdonahue
Copy link
Contributor

Thanks @bhack, see my above comment and then I can merge this.

@bhack
Copy link
Contributor Author

bhack commented Sep 14, 2014

@jeffdonahue Done

jeffdonahue added a commit that referenced this pull request Sep 14, 2014
Add ppa for gflag and glog
@jeffdonahue jeffdonahue merged commit 2da6bc9 into BVLC:dev Sep 14, 2014
mitmul pushed a commit to mitmul/caffe that referenced this pull request Sep 30, 2014
RazvanRanca pushed a commit to RazvanRanca/caffe that referenced this pull request Nov 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants