Update working-with-existing-databases.markdown #262

Merged
merged 1 commit into from Dec 22, 2013

Projects

None yet

4 participants

@ylecuyer

Using build.properties leads to an error:

There was an error building XML from metadata: Unsupported Propel driver: "". Check your configuration file
Schema reverse engineering failed.
@ylecuyer ylecuyer Update working-with-existing-databases.markdown
Using build.properties leads to an error:

```
There was an error building XML from metadata: Unsupported Propel driver: "". Check your configuration file
Schema reverse engineering failed.
```
b6b5ebe
@robin850 robin850 commented on the diff Dec 22, 2013
cookbook/working-with-existing-databases.markdown
- 2. Create a `build.properties` file in `legacyapp/` directory with the DB connection parameters for your existing database, e.g.:
-
-```ini
-propel.project = legacyapp
-
-# The Propel driver to use for generating SQL, etc.
-propel.database = mysql
-
-# This must be a PDO DSN
-propel.database.url = mysql:dbname=legacyapp
-propel.database.user = root
-# propel.database.password #
-```
-
- 3. Run the `reverse` task to generate the `schema.xml`:
+ 2. Run the `reverse` task to generate the `schema.xml`:
@robin850
robin850 Dec 22, 2013

We could add "specifying your database credentials" add the end of the sentence, what do you think ?

@robin850
robin850 Dec 22, 2013

Updated in 4fd2e6d ; thanks @staabm! 😃

@willdurand willdurand merged commit 2cca4bb into propelorm:master Dec 22, 2013

1 check passed

Details default The Travis CI build passed
@willdurand
Propel member

Thanks

@robin850 robin850 added a commit that referenced this pull request Dec 22, 2013
@robin850 robin850 Specify that the user should pass the DB credentials
Follow up of #262
4fd2e6d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment