Currently, cas depends on drupal/cas_mock_server:dev-0.x. The only reason is that cas still supports PHP 5 and drupal/cas_mock_server:dev-0.x is an unofficial branch that supports PHP 5.

Allow cas to pickup drupal/cas_mock_server:dev-1.x when possible.

CommentFileSizeAuthor
#8 3083285-8.patch1.48 KBclaudiu.cristea
#6 3083285-6.patch315 bytesclaudiu.cristea
#5 only-dev-1.x.patch310 bytesclaudiu.cristea
#3 3083285-3.patch428 bytesclaudiu.cristea
#2 3083285-2.patch318 bytesclaudiu.cristea
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

claudiu.cristea created an issue. See original summary.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs work » Needs review
FileSize
3083285-2.patch318 bytes

Patch.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
3083285-3.patch428 bytes

Weird!

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented

Not sure what's going on. I guess the patch from #3 was you trying to fix the composer require failure?

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
only-dev-1.x.patch310 bytes

Debugging.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
3083285-6.patch315 bytes

One more try.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
Assigned: claudiu.cristea » Unassigned

@bkosborne, I have no idea why this is not working. Is it possible that the list of dependencies is built before the patch is applied? At least, this is what #5 is telling us. I'm suggesting that we stop testing with PHP5. Anyway, Drupal 8.7.x supports PHP5 only for existing sites, not for new installs. Any thoughts?

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
3083285-8.patch1.48 KB

Let's patch.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented

I've opened a bug report in DrupalCI: Test Runner: #3112204: Dependency constraint change ignored by CI

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented

I'm going to commit these and see if the tests fail when testing the dev branch.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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