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.

Sign up for GitHub

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

Jump to bottom

Configure where bare repositories get stored #81

Closed
jspahrsummers opened this issue Nov 17, 2014 · 10 comments
Closed

Configure where bare repositories get stored #81

jspahrsummers opened this issue Nov 17, 2014 · 10 comments
Assignees
@robrix
Labels
enhancement update

Comments

@jspahrsummers
Copy link
Member

Defaulting to ~/.carthage/dependencies is reasonable, but let's allow that to be overridden. (Through user defaults, maybe?)
@robrix
Copy link
Contributor

robrix commented Nov 17, 2014

Perhaps off-topic here, but why not ~/Library/Application Support/carthage/dependencies as the default?

@jspahrsummers
Copy link
Member Author

@robrix No real reason—it just seems more idiomatic for command line tools to use a hidden folder in ~ versus Application Support (although I'll definitely grant that the latter is more OS X idiomatic).

@mdiep
Copy link
Member

mdiep commented Nov 17, 2014

I was thinking about this last night and it makes me a bit nervous to have a somewhat hidden and potentially unexpected folder taking up a lot of disk space. 😕

@jspahrsummers
Copy link
Member Author

Pull requests welcome. :trollface:

@jspahrsummers
Copy link
Member Author

Anyways, I think Caches might be more appropriate.

@robrix
Copy link
Contributor

robrix commented Nov 17, 2014

Agreed, Caches.

@alanjrogers
Copy link
Contributor

👍 for Caches.

@robrix robrix mentioned this issue Nov 19, 2014
Merged
5 tasks
@robrix
Copy link
Contributor

robrix commented Nov 19, 2014

Pull requests welcome. :trollface:

#104

@tonyarnold
Copy link
Contributor

Oh god yes please. Don't be that guy. That OS-recommended directory ignoring guy.

@robrix robrix mentioned this issue Nov 24, 2014
Closed
@robb robb mentioned this issue Dec 1, 2014
Closed
@jspahrsummers
Copy link
Member Author

We never did add the customizability piece, but I'm okay with that. Closing this out due to #104.

natanrolnik pushed a commit to natanrolnik/Carthage that referenced this issue Feb 2, 2017
@jpsim
Merge pull request Carthage#81 from jpsim/jp-fix-travis-deploy
5267e86 
fix travis deploy by specifying to run when JOB=Xcode8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@robrix robrix

Labels
enhancement update
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@mdiep @tonyarnold @alanjrogers @robrix @jspahrsummers