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

Magento migration script doesn't honor system paths for wget #2833

Closed
djixas opened this issue Dec 22, 2022 · 2 comments
Closed

Magento migration script doesn't honor system paths for wget #2833

djixas opened this issue Dec 22, 2022 · 2 comments
Labels

Comments

@djixas
Copy link

djixas commented Dec 22, 2022

Preconditions (*)

  1. Installed Magento Commuinity Edition 1.9.4.5
  2. Downloading patch for Magento CE 1.9.4.5 to OpenMage LTS v19.4.8

Steps to reproduce (*)

  1. Run $ curl -fsSL https://migrate.openmage.org | sh import script via user while main server installation has other wget version

Expected result (*)

  1. It should honor paths and not pick defaults

Actual result (*)

  1. wget: unrecognized option '--no-hsts'
    Usage: wget [OPTION]... [URL]...
    Try `wget --help' for more options._

Hi, as you may know, CentOS has older wget which makes openmagelts script not compatible.

I've installed wget as separate bin, but it seems there is no way to "force' a different path for script. Even if I install wget 1.21, OpenMage script does not honor user wget version. It relies on default paths. I did login as user on separate shell account, which gives correct version:

[xx@host ~/public_html]# wget --version
GNU Wget 1.21 built on linux-gnu.

But that does not matter to openmage. The script is simply relying on the system default paths for its dependencies, resulting in wget error since default centos version is 1.14.

@djixas djixas added the bug label Dec 22, 2022
@djixas
Copy link
Author

djixas commented Dec 22, 2022

On the other hand it looks like CentOS may not actually require its own version of wget anyway so I have removed the version provided by CentOS and linked the default path (/bin/wget) to the newer version I installed manually in /opt.

The script is now working in that regard, but it's failing elsewhere when it applies patches to the Magento installation inside the public_html/ directory:

Detected that you have installed Magento Commuinity Edition 1.9.4.5
Performing dry-run patch...
error: patch failed: .htaccess:1
error: .htaccess: patch does not apply
error: .htaccess.sample: No such file or directory
warning: mage has type 100755, expected 100644
Unable to apply the patch. The following command failed:

git --git-dir=/dev/null apply --ignore-whitespace --check var/magento-ce-1.9.4.5-openmage-lts-v19.4.8.patch

However the developers of the migration script should look into this further as it appears to be attempting to apply a specific "openmage" patch.

+--

EDIT:

The last issue was solved by reuploading default .htaccess from magento and htaccess sample!

@addison74
Copy link
Contributor

I am closing this issue. It was reported before in the right place OpenMage/migrate#6.

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

2 participants