For the git_wrapper module to be installed, git_wrapper_requirements() wants the cpliakas/git-wrapper composer package installed in the git_wrapper module directory so that file_exists(__DIR__ . '/vendor/autoload.php')

However, then the packages don't appear under admin/config/system/composer-manager/packages

When I have composer-manager build its aggregate composer.json file and run install from the 'Composer File Directory' I end up with another installation of the composer packages in composer-manager's specified location.

I would like to be able to just install all composer packages at sites/all/libraries/composer - or maybe I'm not fully understanding.

CommentFileSizeAuthor
#2 requirements-1935562-1.patch2.48 KBcpliakas

Comments

cpliakas’s picture

cpliakas commented
Category: support » bug
Priority: Normal » Critical

Hi ericras.

Thanks for picking this up. This is actually a critical bug, as the requirements are left over from before Composer Manager was a dependency. As a result, the logic is out of sync with how things should work. You are correct in that all composer packages should be managed in a single location, the default being sites/all/libraries/composer.

I will roll a patch with a corresponding RC2 release immediately.

Thanks,
Chris

cpliakas’s picture

cpliakas commented
Title: Where should git-wrapper composer package be installed? » Outdated requirements expect autoloader in the wrong location
Component: Documentation » Code
Status: Active » Needs review
StatusFileSize
new requirements-1935562-1.patch2.48 KB

Attached patch removes the outdated requirements and fix some other issues as a result of moving to Composer Manager to manage the packages.

cpliakas’s picture

cpliakas commented
Status: Needs review » Fixed

Committed at http://drupalcode.org/project/git_wrapper.git/commit/5b69405.

cpliakas’s picture

cpliakas commented

Change reflected in the RC2 release.

ericras’s picture

ericras commented

Installs perfect now. I'll be playing around with git-sync.

cpliakas’s picture

cpliakas commented

ericras,

Thanks for reporting back. Git Sync is very beta, so I look forward to you breaking that as well ;-)

Good luck,
Chris

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.