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

Credis symlinks wrong? #2781

Closed
sreichel opened this issue Dec 4, 2022 · 5 comments
Closed

Credis symlinks wrong? #2781

sreichel opened this issue Dec 4, 2022 · 5 comments
Assignees
Labels

Comments

@sreichel
Copy link
Contributor

sreichel commented Dec 4, 2022

Preconditions (*)

  1. latest 19.4.x (with composer modules added)

Steps to reproduce (*)

  1. added this to composer.json
    "magento-deploystrategy": "copy",
    "magento-deploystrategy-dev": "symlink",
    "magento-force": true,
  1. run composer install ... symlinks for CM_ are created
  2. delete vendor
  3. run composer install --no-dev

Expected result (*)

  1. symlinks should be replaced by files

Actual result (*)

  1. Credis copy fails

credis
credis2

@fballiano
Copy link
Contributor

when I do the composer install --no-dev I get:

In Copy.php line 133:
                                                                                                                                                                                                                                                           
  copy(/Users/fab/Projects/openmage/vendor/colinmollenhour/cache-backend-redis/lib/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis/Credis): failed to open stream: Invalid argument   

I think the problem is with the "magento-force" option

@sreichel
Copy link
Contributor Author

sreichel commented Dec 6, 2022

colinmollenhour/Cm_Cache_Backend_Redis#65 is not fixed lib/Credis is empty.

@sreichel sreichel mentioned this issue Dec 10, 2022
4 tasks
@AGelzer
Copy link
Contributor

AGelzer commented Jan 14, 2023

Info: magento-composer-installer has problems with change deploy strategy from symlink to copy

See: Pull request #208

@sreichel
Copy link
Contributor Author

sreichel commented Jan 15, 2023

@AGelzer thank you.

@Flyingmana your're maintainer there ... do you have time to review this?

@Flyingmana
Copy link
Contributor

Flyingmana commented Jan 15, 2023

For this I first need to know if the same happens when using modman, as we tried back then to copy the behavior (which included bugs like these if directories were already existing)
Also changing without removing is more or less undefined behaviour.

EDIT:
checked more in detail. The removal of symlinks/files happens only during uninstall of the module, not when vendor is deleted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants