Problem/Motivation
When test installing the dependencies on gets
$ composer install
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for clue/graph-uml dev-master -> satisfiable by clue/graph-uml[dev-master].
- clue/graph-uml dev-master requires clue/graph 0.7.* -> satisfiable by clue/graph[v0.7.0, v0.7.1] but these conflict with your requirements or minimum-stability.
Reading https://www.drupal.org/docs/8/creating-custom-modules/add-a-composerjson... as a developer we need to require this library on Drupal level
sites/drupal/d8/www % composer require clue/graph-uml
Proposed resolution
Use the correct libraries from https://github.com/clue/graph and/or https://github.com/graphp. See also https://github.com/clue/graph/issues/130
Remaining tasks
- [x] Find out which pinned versions to use instead of dev-master
- [ ] Remove class_loader stuff as we have to use Drupal level class loader according to https://www.drupal.org/docs/8/creating-custom-modules/add-a-composerjson...
Comments
Comment #2
clemens.tolboomYeah ... I'll look into that soon as I worked on that code a little too.
Comment #3
clemens.tolboomComment #4
clemens.tolboomReading https://www.drupal.org/docs/8/creating-custom-modules/add-a-composerjson... and https://www.drupal.org/node/2405811 it seems like project users have to install Drupal AND GraphAPI through composer all the way.
Checking our composer.json I have no clue why some requirements are added
For us developer we have to manually add "clue/graph": "dev-master" and others but I know lots of changes and code moves are done in that project so this needs more work :-/
Comment #5
clemens.tolboomComment #6
clemens.tolboomgit blame composer.jsongivesComment #8
clemens.tolboomSo we can fix this for now by let graph-uml decide on version.
This is now kinda fixed. When rendering starts working we need to get back to this I guess.
Comment #9
clemens.tolboomI had to
Comment #10
clemens.tolboomThis is not working as expected.
Changing Drupal composer.json makes the composer.lock file out of date.
Running
composer installon graphapi directory makes it's class loader work.Comment #11
joachim commentedI just did a git pull, and composer install worked:
Comment #12
joachim commentedWhoops, didn't mean to change status!
Comment #13
clemens.tolboomIt works but not as documented by d.o ... we'll live for now so not critical.
Comment #14
clemens.tolboomI cannot get demo pages to work http://drupal.d8/admin/config/system/graphapi/formats
$text .= 'See ' . l("$project_id", 'http://drupal.org/project/' . $project_id) . ' on drupal.org';Temp fixing both and install the remaining dependencies through root composer as the provided dependencies do not work.
does not help as
which makes me wonder how many code changes we need to make it work again.
Comment #15
joachim commented