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

#26 Make all fails #27

Merged
merged 1 commit into from
Jan 14, 2021
Merged

#26 Make all fails #27

merged 1 commit into from
Jan 14, 2021

Conversation

tigokr
Copy link
Member

@tigokr tigokr commented Jan 14, 2021

  • fixed and improved Makefile

- fixed and improved Makefile
@tigokr tigokr requested a review from stepanets January 14, 2021 05:55
@tigokr tigokr self-assigned this Jan 14, 2021
@stepanets
Copy link
Member

stepanets commented Jan 14, 2021

docker run --rm -u=1000:1000 -v /hdd/projects/elegant-bro/money:/app -w /app elegant-bro/money:7.4 composer install
Cannot create cache directory /.composer/cache/repo/https---repo.packagist.org/, or directory is not writable. Proceeding without cache
Cannot create cache directory /.composer/cache/files/, or directory is not writable. Proceeding without cache
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Warning: The lock file is not up to date with the latest changes in composer.json. You may be getting outdated dependencies. It is recommended that you run `composer update` or `composer update <package name>`.
Package operations: 13 installs, 43 updates, 38 removals
  - Downloading elegant-bro/interfaces (1.1.1)
  - Downloading elegant-bro/interfaces (1.1.1)
  - Downloading elegant-bro/interfaces (1.1.1)
  - Downloading elegant-bro/interfaces (1.1.1)
    Failed to download elegant-bro/interfaces from dist: The "https://api.github.com/repos/elegant-bro/interfaces/zipball/005e9e22dc77ea21d80c0fc358d12a901f4e59f0" file could not be written to /app/vendor/composer/tmp-a047ca02f1ba08c98720ce2e46937cb4: failed to open stream: Permission denied

Looks like composer cache doesn't mapped to the host.

And what about Failed to download elegant-bro/interfaces from dist:?

@tigokr
Copy link
Member Author

tigokr commented Jan 14, 2021

you should remove before your vendor folder
docker run now executed with non-privileged user this can tends to some "Permission denied" errors with "not fresh" repo

Copy link
Member

@stepanets stepanets left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome job, bro!

@stepanets stepanets merged commit dca9aea into master Jan 14, 2021
@stepanets stepanets deleted the fix-#26_makefile branch January 14, 2021 06:20
@stepanets stepanets linked an issue Jan 14, 2021 that may be closed by this pull request
@stepanets stepanets added the bug Something isn't working label Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make all fails
2 participants