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

Update symfony/flex. #144

Merged
merged 2 commits into from
Apr 7, 2022
Merged

Update symfony/flex. #144

merged 2 commits into from
Apr 7, 2022

Conversation

jordandukart
Copy link
Member

GitHub Issue: Islandora/documentation#2079

What does this Pull Request do?

Updates symfony/flex that's required due to an end-of-life of their previous download manager.

What's new?

Version bump.

How should this be tested?

Houdini should still operate as expected.

Interested parties

@alxp

@codecov
Copy link

codecov bot commented Apr 6, 2022

Codecov Report

Merging #144 (ac852df) into 2.x (75ddd81) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.x     #144   +/-   ##
=========================================
  Coverage     76.60%   76.60%           
  Complexity      161      161           
=========================================
  Files             6        6           
  Lines           654      654           
=========================================
  Hits            501      501           
  Misses          153      153           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 75ddd81...ac852df. Read the comment docs.

@alxp
Copy link
Contributor

alxp commented Apr 6, 2022

Still seem to be getting the same error on composer install and also update:

vagrant@islandora8:/var/www/html/Crayfish/Houdini$ composer install
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.

In PackageFilter.php line 125:
                           
  Undefined index: splits  
                           

install [--prefer-source] [--prefer-dist] [--prefer-install PREFER-INSTALL] [--dry-run] [--dev] [--no-suggest] [--no-dev] [--no-autoloader] [--no-progress] [--no-install] [-v|vv|vvv|--verbose] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--apcu-autoloader-prefix APCU-AUTOLOADER-PREFIX] [--ignore-platform-req IGNORE-PLATFORM-REQ] [--ignore-platform-reqs] [--] [<packages>...]

vagrant@islandora8:/var/www/html/Crayfish/Houdini$ git branch
  2.x
* 2.x-ISLANDORA-2079
  dev
vagrant@islandora8:/var/www/html/Crayfish/Houdini$ 

@alxp
Copy link
Contributor

alxp commented Apr 6, 2022

The GitHub page referencing this error mentions that symfony/flex should be at least version 1.17.

On this PR's branch it's at 1.13.

$ composer info symfony/flex
name     : symfony/flex
descrip. : Composer plugin for Symfony
keywords : 
versions : * v1.13.3
type     : composer-plugin
license  : MIT License (MIT) (OSI approved) https://spdx.org/licenses/MIT.html#licenseText
homepage : 
source   : [git] https://github.com/symfony/flex.git 2597d0dda8042c43eed44a9cd07236b897e427d7
dist     : [zip] https://api.github.com/repos/symfony/flex/zipball/2597d0dda8042c43eed44a9cd07236b897e427d7 

@jordandukart
Copy link
Member Author

Thanks for catching @alxp derped on composer update versus composer require.

@alxp
Copy link
Contributor

alxp commented Apr 7, 2022

Updated and tested on fresh Playbook install.

Was able to see Service File mp4, so this update does not seem break Crayfish's functionality.

image

@alxp alxp merged commit fd4284b into Islandora:2.x Apr 7, 2022
whikloj added a commit to whikloj/Crayfish that referenced this pull request Sep 16, 2022
whikloj pushed a commit to whikloj/Crayfish that referenced this pull request Sep 16, 2022
* Update symfony/flex.

* Actually update the require.
seth-shaw-asu pushed a commit that referenced this pull request Sep 19, 2022
* Update symfony/flex.
Co-authored-by: Jordan Dukart <jordan@discoverygarden.ca>
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.

2 participants