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 #39

Merged
merged 14 commits into from
May 4, 2020
Merged

Update #39

merged 14 commits into from
May 4, 2020

Conversation

lasseborly
Copy link
Contributor

@lasseborly lasseborly commented Apr 8, 2020

This primarily serves as an readme update but also has a couple other tricks up it's sleeve.

With the changes the last couple of years the readme has gotten little love and has fallen behind.

Try to follow the steps in the readme and if something isn't working please do alarm me here.

@lasseborly lasseborly self-assigned this Apr 8, 2020
@rasben
Copy link
Member

rasben commented Apr 14, 2020

I fail on make up:

  - Installing swiftmailer/swiftmailer (v5.0.3): Loading from cache
  - Installing symfony/swiftmailer-bundle (v2.3.5): Loading from cache
  - Installing twig/extensions (v1.0.1): Loading from cache
jms/serializer-bundle suggests installing jms/di-extra-bundle (Required to get lazy loading (de)serialization visitors, ~1.3)
kriswallsmith/assetic suggests installing leafo/lessphp (Assetic provides the integration with the lessphp LESS compiler)
kriswallsmith/assetic suggests installing leafo/scssphp (Assetic provides the integration with the scssphp SCSS compiler)
kriswallsmith/assetic suggests installing leafo/scssphp-compass (Assetic provides the integration with the SCSS compass plugin)
kriswallsmith/assetic suggests installing ptachoire/cssembed (Assetic provides the integration with phpcssembed to embed data uris)
monolog/monolog suggests installing aws/aws-sdk-php (Allow sending log messages to AWS services like DynamoDB)
monolog/monolog suggests installing doctrine/couchdb (Allow sending log messages to a CouchDB server)
monolog/monolog suggests installing ext-amqp (Allow sending log messages to an AMQP server (1.0+ required))
monolog/monolog suggests installing ext-mongo (Allow sending log messages to a MongoDB server)
monolog/monolog suggests installing mlehner/gelf-php (Allow sending log messages to a GrayLog2 server)
monolog/monolog suggests installing raven/raven (Allow sending log messages to a Sentry server)
monolog/monolog suggests installing ruflin/elastica (Allow sending log messages to an Elastic Search server)
Package symfony/icu is abandoned, you should avoid using it. Use symfony/intl instead.
Generating autoload files
> Incenteev\ParameterHandler\ScriptHandler::buildParameters
Updating the "app/config/parameters.yml" file
Deprecation Notice: The callback Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::buildBootstrap declared at /harvester/vendor/sensio/distribution-bundle/Sensio/Bundle/DistributionBundle/Composer/ScriptHandler.php accepts a Composer\Script\CommandEvent but post-install-cmd events use a Composer\Script\Event instance. Please adjust your type hint accordingly, see https://getcomposer.org/doc/articles/scripts.md#event-classes in phar:///usr/local/bin/composer/src/Composer/EventDispatcher/EventDispatcher.php:339
> Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::buildBootstrap
Deprecation Notice: The callback Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache declared at /harvester/vendor/sensio/distribution-bundle/Sensio/Bundle/DistributionBundle/Composer/ScriptHandler.php accepts a Composer\Script\CommandEvent but post-install-cmd events use a Composer\Script\Event instance. Please adjust your type hint accordingly, see https://getcomposer.org/doc/articles/scripts.md#event-classes in phar:///usr/local/bin/composer/src/Composer/EventDispatcher/EventDispatcher.php:339
> Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache
Clearing the cache for the dev environment with debug true



  [Symfony\Component\Filesystem\Exception\IOException]
  Cannot rename "/harvester/app/cache/dev" to "/harvester/app/cache/dev_old".



cache:clear [--no-warmup] [--no-optional-warmers]


Script Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache handling the post-install-cmd event terminated with an exception


  [RuntimeException]
  An error occurred when executing the "'cache:clear --no-warmup'" command.


install [--prefer-source] [--prefer-dist] [--dry-run] [--dev] [--no-dev] [--no-custom-installers] [--no-autoloader] [--no-scripts] [--no-progress] [--no-suggest] [-v|vv|vvv|--verbose] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--ignore-platform-reqs] [--] [<packages>]...

make: *** [up] Error 1

@rasben
Copy link
Member

rasben commented Apr 14, 2020

Her er de filer som jeg har, og min app/cache mappe er tom:

-rw-r--r--   1 rasben  staff   2.0K Apr 14 14:57 Dockerfile
-rw-r--r--   1 rasben  staff   1.0K Apr 14 14:57 LICENSE
-rw-r--r--   1 rasben  staff   365B Apr 14 14:58 Makefile
-rw-r--r--   1 rasben  staff   2.3K Apr 14 14:58 README.md
-rw-r--r--   1 rasben  staff   1.3K Apr 14 14:57 UPGRADE-2.2.md
-rw-r--r--   1 rasben  staff   1.9K Apr 14 14:57 UPGRADE-2.3.md
-rw-r--r--   1 rasben  staff   356B Apr 14 14:57 UPGRADE-2.4.md
-rw-r--r--   1 rasben  staff   8.3K Apr 14 14:57 UPGRADE.md
drwxr-xr-x  14 rasben  staff   448B Apr 14 14:57 app
-rw-r--r--   1 rasben  staff   2.5K Apr 14 14:57 composer.json
-rw-r--r--   1 rasben  staff    81K Apr 14 14:57 composer.lock
drwxr-xr-x   2 rasben  staff    64B Apr 14 15:04 data
drwxr-xr-x  10 rasben  staff   320B Apr 14 14:57 docker
-rw-r--r--   1 rasben  staff   302B Apr 14 15:00 docker-compose.override.yml
-rw-r--r--   1 rasben  staff   313B Apr 14 14:57 docker-compose.override.yml-example
-rw-r--r--   1 rasben  staff   279B Apr 14 14:58 docker-compose.yml
-rw-r--r--   1 rasben  staff   333B Apr 14 14:57 phpci.yml
drwxr-xr-x   4 rasben  staff   128B Apr 14 14:57 src
drwxr-xr-x  23 rasben  staff   736B Apr 14 15:07 vendor
drwxr-xr-x   8 rasben  staff   256B Apr 14 14:57 web
╭─rasben@Benjamins-MBP ~/code/harvester ‹chore/update-readme›
╰─$ ll app/cache

@rasben
Copy link
Member

rasben commented Apr 14, 2020

og det undrer mig at der ikke er nogen container der kører:


╰─$ dco ps                                                                                                                                                                                                               
Name   Command   State   Ports
------------------------------

Har den nu lavet en masse gøgl på min lokale maskine? :o

@lasseborly
Copy link
Contributor Author

og det undrer mig at der ikke er nogen container der kører:


╰─$ dco ps                                                                                                                                                                                                               
Name   Command   State   Ports
------------------------------

Har den nu lavet en masse gøgl på min lokale maskine? :o

Kør lige dco ps -a og se om den ikke har kørt.

@rasben
Copy link
Member

rasben commented Apr 14, 2020

╭─rasben@Benjamins-MBP ~/code/harvester ‹chore/update-readme›
╰─$ dco ps -a
                Name                       Command        State    Ports
------------------------------------------------------------------------
harvester_harvester_run_ec9abf2a421c   composer install   Exit 1

@lasseborly
Copy link
Contributor Author

Cannot rename "/harvester/app/cache/dev" to "/harvester/app/cache/dev_old".

Hmmm lidt mere omfattende end som så for macos..

symfony/symfony#2600 (comment)

This primarily makes talking about getting started easier.
We make use of the harvester image that has composer installed instead.
@lasseborly lasseborly changed the title Update readme Update May 3, 2020
This is done to match the actual usage better and to match the upcoming timelord naming scheme better.
This is to prevent permissions errors on macOS.
We no longer use this and having it at root lead to a bit of confusion.
Was not being used.
This was never used and never checked. Remove it for clarity.
This update reflects some of the changes that has been done to the code base along the last couple of years.
Also it renames the service to better in accordance with Timelord.
Copy link
Member

@rasben rasben left a comment

Choose a reason for hiding this comment

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

It works on mac 👍

@lasseborly lasseborly merged commit 812e4c3 into master May 4, 2020
@lasseborly lasseborly deleted the chore/update-readme branch May 4, 2020 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants