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

Package type common-webroot #53

Closed
wants to merge 1 commit into from
Closed

Conversation

sbuzonas
Copy link

@sbuzonas sbuzonas commented Jan 8, 2013

Proposal for webroot themed packages. In #38 drupal-core places the package in core. This commit exposes a new extra in the inflected vars {$webroot} that can be used by packages. It forces installer-webroot to be defined on the root package for a common-webroot package.

This proposes a package to be defined as a webroot application with a variable webroot location. A way to assert that there is only one package of that type would be ideal, not quite sure how to implement that.

if (!empty($extra['installer-paths'])) {
$customPath = $this->mapCustomInstallPaths($extra['installer-paths'], $prettyName);
if ($customPath !== false) {
return $this->templatePath($customPath, $availableVars);
}
}
} elseif ('common-webroot' === $type) {
Copy link
Author

Choose a reason for hiding this comment

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

It may be better to implement a hook for installers to validate package configuration instead of adding this into base installer here.

@shama
Copy link
Contributor

shama commented Jan 9, 2013

Thanks Steve! I'll give this some thought and a look over soon. I really want to implement #43 before moving on this type of thing. Just to make sure installers continues to work appropriately with composer.

@sbuzonas
Copy link
Author

sbuzonas commented Feb 4, 2013

I created a separate installer project to cover this case. I intend to have an overlay concept on the roadmap. https://github.com/fancyguy/webroot-installer

Not quite sure which approach is better, what is the overall goal of composer/installers? To handle common framework extensions or be a general purpose catch-all installer?

@shama
Copy link
Contributor

shama commented Feb 4, 2013

Awesome thanks! I'll direct people there if they are also looking for this use case.

I see composer/installers as merely a shim for frameworks that one day, hopefully, can be deprecated.

@shama shama closed this Feb 4, 2013
@davidbarratt
Copy link
Contributor

Isn't this what create-project is for?

http://getcomposer.org/doc/03-cli.md#create-project

thanks!

@sbuzonas sbuzonas mentioned this pull request Jun 4, 2013
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.

None yet

3 participants