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

Use packages.drupal.org repository #159

Closed
wants to merge 1 commit into from

Conversation

oxyc
Copy link

@oxyc oxyc commented Jun 6, 2016

Or is it still bleeding edge?

@ryanaslett
Copy link

its not totally bleeding edge, but theres a couple of rough patches here n there.

@geerlingguy
Copy link

Platform.sh just announced they're defaulting to the new repository, so it seems like it may be stable enough for people to start using it in the wild (and helping smooth the rough patches!).

@ryanaslett
Copy link

Im probably being overly critical of the theoretical rough edges. The main outstanding issue is this: https://www.drupal.org/node/2744539, predicated by this: https://www.drupal.org/node/2744455.

So even though its only one module, there's about 17 other modules with a reference to media_entity as a dependency, if we extract that out further to see how many modules depend on media_entity through some strange dependency chain, it leaves just enough room for me to call it a 'rough edge'

@shorock
Copy link

shorock commented Jun 9, 2016

Working well for me on D8, and noticeably faster.

Thinking D7, the packages.drupal.org/7 endpoint doesn't provide the synthesized (for lack of a better term) drupal/drupal:7.* jsons that packagist.drupal-composer.org does. Seems to provide just about everything else for 7. I haven't been following the discussion enough to know if that's in the works for the drupal.org endpoints.

For interim testing in anticipation of a better answer, in my composer-built D7 projects, I've been doing...

  "repositories": [{
    "type": "composer",
    "url": "https://packages.drupal.org/7"
  },
  {
    "type": "composer",
    "url": "https://packagist.drupal-composer.org"
  }],

to prefer the drupal.org but fall back.

@ryanaslett
Copy link

@shorock , could you elaborate on the d7 issue? According to the logfiles, I havent seen anybody even use the d7 composer endpoint yet, so Im curious if something is just broken or what.

drupal/drupal should be found at packagist.org, so as to prevent chicken/egg scenarios where composer create-project drupal/drupal doesnt know to go to the packages.drupal.org endpoint.

@fafnirical
Copy link

One breaking change in switching to the Drupal.org Composer Service is that distributions and install profiles aren't yet supported: https://www.drupal.org/node/2715505

@shorock
Copy link

shorock commented Jun 10, 2016

@ryanaslett - I was guessing that I was pretty early on it, but I've been building things in the @Islandora world (includes a whole darned lot of github submodules, very nice to use composer) using the p.d.o/7 endpoint for a couple weeks now.

The packagist.org/packages/drupal/drupal doesn't have 7.* builds. The deprecating drupal-composer packagist does.

Anyway, just mentioning the 7.x branch in passing. Don't let it block the 8.x PR, though both the noted concerns above are worth considering.

@webflo
Copy link
Member

webflo commented Jun 17, 2016

Related issue drupal-composer/drupal-packagist#61

@shorock
Copy link

shorock commented Jun 24, 2016

Just an update for those who don't do D7 Composer that quite a bit has changed (positively) in a fortnight. The packages.drupal.org/7 endpoint now returns drupal/drupal:7.* packages (although, interestingly, only to 7.43, not 7.44... will check/report there). Packagist.org/packages/drupal/drupal also now has the 7.* packages that it didn't have two weeks back.

@ryanaslett
Copy link

Not sure why 7.44 wasn't there, the timestamp of the file coincided with the release. I've re-ran the metadata builder and 7.44 is there now. I'll open an issue to remind myself to investigate if there is some sort of race condition happening on new releases.

@ryanaslett
Copy link

Also, I dont think that d7 drupal/drupal exists on packagist: https://packagist.org/p/drupal/drupal%2466ae3a5c3958ab51c9a19bdc5a36b45f55d1ae2a0e82857e79ad8855a5f3822b.json is the current provider file, and a composer info drupal/drupal --all without the d.o. repo only shows the 8.x versions.

@webflo
Copy link
Member

webflo commented Jul 12, 2016

@oxyc I think we could keep the drupal/drupal conflict. Because d.o switched to drupal/core as base package. Or we remove it completely?

@webflo
Copy link
Member

webflo commented Oct 4, 2016

Thanks! Committed to 8.x in 49184d6

@webflo webflo closed this Oct 4, 2016
shrop pushed a commit to shrop/drupal-project that referenced this pull request Apr 11, 2020
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

6 participants