Skip to content
This repository has been archived by the owner on Nov 27, 2020. It is now read-only.

Move Acme demo bundle to sensio DistributionBundle #643

Merged
merged 1 commit into from
Mar 30, 2014

Conversation

romainneutron
Copy link
Contributor

@romainneutron
Copy link
Contributor Author

How to test this ?

First,

git clone git@github.com:romainneutron/symfony-standard.git

then edit composer.json and add:

    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/romainneutron/SensioDistributionBundle"
        }
    ],
    "require": {
        "sensio/distribution-bundle": "dev-acme-demo-bundle"
    }

finally

composer install

@romainneutron
Copy link
Contributor Author

Weird error, fabbot propose me to patch a file that bas been removed. Here is the proposed patch:

--- src/Acme/DemoBundle/Controller/WelcomeController.php    2014-03-20 17:31:33.845442703 +0000
+++ src/Acme/DemoBundle/Controller/WelcomeController.php    2014-03-20 17:31:39.109442703 +0000
@@ -13,6 +13,7 @@
          * or @Template annotation as demonstrated in DemoController.
          *
          */
+
         return $this->render('AcmeDemoBundle:Welcome:index.html.twig');
     }
 }

fabpot added a commit that referenced this pull request Mar 30, 2014
…inneutron)

This PR was merged into the 2.4-dev branch.

Discussion
----------

Move Acme demo bundle to sensio DistributionBundle

See https://github.com/symfony/symfony-standard/issues/639
Merge sensiolabs/SensioDistributionBundle#119 before this one

Commits
-------

eb5d496 Move Acme demo bundle to sensio DistributionBundle
@fabpot fabpot merged commit eb5d496 into symfony:master Mar 30, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants