Skip to content
This repository has been archived by the owner on May 7, 2019. It is now read-only.

Add a directory for each application #89

Merged
merged 11 commits into from
Oct 23, 2015

Conversation

poupotte
Copy link
Contributor

Directory is located in /usr/local/var/cozy/<app>

@clochix
Copy link
Contributor

clochix commented Oct 15, 2015

Awesome !!!
I suppose we should add the ability to delete this directory (for example with cozy-monitor uninstall --purge)

@frankrousseau
Copy link
Contributor

@poupotte @clochix What we should do with that PR? Wait for the command?

Whatever it requires a rebase.

initAppsDir = (callback) ->
apps = fs.readdirSync config('dir_source')
async.forEach apps, (app, cb) ->
if app is 'stack.json'
Copy link
Contributor

Choose a reason for hiding this comment

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

A comment explaining this condition would be nice.

@clochix
Copy link
Contributor

clochix commented Oct 19, 2015

@poupotte opened an issue in cozy-monitor to add the purge option. I think we can already merge this one.

@frankrousseau
Copy link
Contributor

Ok let's wait for @poupotte rebase.

Create directory for <app>
###
module.exports.create = (app, callback) ->
changeOwner = (path, callback) ->
Copy link
Contributor

Choose a reason for hiding this comment

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

@poupotte if you can move this function into another function at the root level of the module, it would be cleaner.

@clochix
Copy link
Contributor

clochix commented Oct 23, 2015

Hello everybody, what's the status of this awesome and much awaited work ?

frankrousseau pushed a commit that referenced this pull request Oct 23, 2015
Add a directory for each application
@frankrousseau frankrousseau merged commit 25b7e60 into cozy:development Oct 23, 2015
@frankrousseau
Copy link
Contributor

Let's merge it. Thank you @poupotte, @nono and @clochix.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants