Skip to content

Commit

Permalink
Update composer.json (#531)
Browse files Browse the repository at this point in the history 
Copy the dbcopy.php also on "composer update", not only on "composer install".
  • Loading branch information
@J-A-U @andig
J-A-U authored and andig committed Dec 8, 2016
1 parent 8f92692 commit efb1213
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion composer.json
View file
Expand Up @@ -45,6 +45,6 @@
}
],
"scripts": {
"post-install-cmd": "php -r \"@copy('vendor/andig/dbcopy/dbcopy.php', 'misc/tools/dbcopy.php');\""
"post-update-cmd": "php -r \"@copy('vendor/andig/dbcopy/dbcopy.php', 'misc/tools/dbcopy.php');\""
}
}

4 comments on commit efb1213

@justinotherguy
Copy link
Member

Choose a reason for hiding this comment

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

Moin,
this commit seems to cause my "/usr/local/bin/composer update" to write this to my console:

php -r "@copy('vendor/andig/dbcopy/dbcopy.php', 'misc/tools/dbcopy.php');"

I wonder:

  • does this mean, the command is executed?
  • if I delete misc/tools/dbcopy.php before executing "composer update", the file is copied (I get the output in either case); so at least in case the target does not exist, it is being executed
  • remains the question: is there a specific reason for writing this to stdout?

Regards, J.

@andig
Copy link
Contributor

@andig andig commented on efb1213 Dec 11, 2016 via email

Choose a reason for hiding this comment

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

@justinotherguy
Copy link
Member

@justinotherguy justinotherguy commented on efb1213 Dec 11, 2016 via email

Choose a reason for hiding this comment

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

@andig
Copy link
Contributor

@andig andig commented on efb1213 Dec 11, 2016 via email

Choose a reason for hiding this comment

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

Please sign in to comment.