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

Github Download Support #12

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Github Download Support #12

wants to merge 29 commits into from

Conversation

ngtjah
Copy link

@ngtjah ngtjah commented Dec 15, 2015

I wanted to get the latest version of s3fs-fuse and found that the install recipe was pointed to googlecode.com while they have moved their current repository to github. So I've made two changes.

  1. Update the download link for versions that are >= 1.74 (when it was moved).
  2. Added the ./autogen.sh at v1.74 although it is probably used before this version.

-ngtjah

s3fs_version = node[:s3fs_fuse][:version]
source_url = "http://s3fs.googlecode.com/files/s3fs-#{s3fs_version}.tar.gz"
s3fs_git_tag = node[:s3fs_fuse][:version]
s3fs_version = node[:s3fs_fuse][:version].tr('^0-9.','')
Copy link
Member

Choose a reason for hiding this comment

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

Can you provide a little more background why this change was needed?

@webframp
Copy link
Member

This could be much simplified if we just decide to deprecate support for s3fs-fuse older than 1.74. Since that version was released on Github Nov 25, 2013 that seems like a reasonable choice to me. Adding a note to the readme/changelog should be sufficient.

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.

2 participants