In composer require-dev many modules are required.

  "require-dev": {
    "drupal/address": "~1.4",
    "drupal/bootstrap": "~3.0",
    "drupal/captcha": "~1.0",
    "drupal/chosen": "~2.4",
    "drupal/entity_print": "^2.1",
    "drupal/group": "~1.0",
    "drupal/mailsystem": "~4.1",
    "drupal/select2": "~1.1",
    "drupal/smtp": "~1.0",
    "drupal/telephone_validation": "^2.2",
    "drupal/token": "~1.3"
  }

The version numbers of these modules are going out of date and are quite restrictive, which makes it very difficult for composer to resolve stuff without the --no-dev flag.

CommentFileSizeAuthor
#15 3324822-15.patch1.91 KBhmendes
#12 interdiff_6-11.txt1.06 KBlucienchalom
#12 interdiff_10-11.txt2.18 KBlucienchalom
#12 3324822-11.patch2.01 KBlucienchalom
#10 3324822-10.patch2.08 KBWagnerMelo
#10 interdiff-9-10.txt1.14 KBWagnerMelo
#9 interdiff-6-9.txt2.17 KBWagnerMelo
#9 3324822-9.patch2.08 KBWagnerMelo
#6 3324822-6.patch1.99 KBjrockowitz
#4 3324822-4.patch1.88 KBjrockowitz

Comments

matslats created an issue. See original summary.

jrockowitz’s picture

jrockowitz commented

Okay, let's update these version numbers and make them more flexible.

jrockowitz’s picture

jrockowitz commented
Status: Active » Needs work
jrockowitz’s picture

jrockowitz commented
Status: Needs work » Needs review
StatusFileSize
new 3324822-4.patch1.88 KB

Let's try using the latest dev versions.

matslats’s picture

matslats commented

that's exactly what I had in mind thanks.
Not sure what there is to review here.

jrockowitz’s picture

jrockowitz commented
Version: 6.2.x-dev » 6.1.x-dev
StatusFileSize
new 3324822-6.patch1.99 KB
jrockowitz’s picture

jrockowitz commented
Status: Needs review » Needs work

I have to test these dev dependency combinations locally.

WagnerMelo’s picture

WagnerMelo commented
Assigned: Unassigned » WagnerMelo

Hi, i'll try work in this issue. XD

WagnerMelo’s picture

WagnerMelo commented
Assigned: WagnerMelo » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 3324822-9.patch2.08 KB
new interdiff-6-9.txt2.17 KB

HI, i worked in this issues, and the patch sended by @jrockowitz at #6 look like that work correctly.
I tried do some test in my local enviroment and everything work ok, all dependencies was downloaded right.
I only notice a stuff, i look in other composer.json file from other modules, and when it requires some modules with the flag -dev, there was writte following this pattern, for exemple "drupal/devel": "5.x-dev@dev". So i'll send a new patch for review.
I expect helped in this issues.

WagnerMelo’s picture

WagnerMelo commented
StatusFileSize
new interdiff-9-10.txt1.14 KB
new 3324822-10.patch2.08 KB

Hi, i'm still working in this issue, to try solve the tests failures.
I found some mistakes on patch, so i'll send other patch to see if we get another result in tests.

WagnerMelo’s picture

WagnerMelo commented
Status: Needs review » Needs work
lucienchalom’s picture

lucienchalom commented
Status: Needs work » Needs review
StatusFileSize
new 3324822-11.patch2.01 KB
new interdiff_10-11.txt2.18 KB
new interdiff_6-11.txt1.06 KB

I tried to change bartik and seven to alpha releases to see if they now match the constrain for dev.
also used from patch #6 and patch #10, so added an interdiff for both.

Status: Needs review » Needs work

The last submitted patch, 12: 3324822-11.patch, failed testing. View results

jrockowitz’s picture

jrockowitz commented

This looks pretty close, and we need to figure out the Bartik css issue.

hmendes’s picture

hmendes commented
Status: Needs work » Needs review
StatusFileSize
new 3324822-15.patch1.91 KB

I don't understand why we added bartik and seven to this composer.json, removing it to see if the tests pass.
If those 2 lines are needed, please ignore this patch (and if someone can explain to me why to use them I would appreciate it)

jrockowitz’s picture

jrockowitz commented

Bartik and Seven are needed for testing Drupal 10.0.x but we might be able to remove that dependency/

elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented

Hi I was able to install the dependencies putting them in composer.json at the root of directory.

I ran lando composer install the installation was cleanly but I don't know if it's enough to switch to RTBC

I'm using drupal 9.5 version.

jrockowitz’s picture

jrockowitz commented
Status: Needs review » Reviewed & tested by the community

  • hmendes authored 4b4ed291 on 6.1.x 
    Issue #3324822 by WagnerMelo, jrockowitz, lucienchalom, hmendes:...

  • hmendes authored 4b4ed291 on 6.x 
    Issue #3324822 by WagnerMelo, jrockowitz, lucienchalom, hmendes:...

  • hmendes authored 4b4ed291 on 6.2.x 
    Issue #3324822 by WagnerMelo, jrockowitz, lucienchalom, hmendes:...
jrockowitz’s picture

jrockowitz commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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