loiclacombe / dbdeploy

Automatically exported from code.google.com/p/dbdeploy
0 stars 0 forks source link

Trimming change scripts can make e.g. views and SPs unreadable in vendor supplied tools #55

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
When in "apply direct to db mode", dbdeploy trims leading and trailing 
whitespace from every line before supplying it to the database. 

I can't quite remember why it did this, but I think it was to improve the 
detection of non-row statement separators.

However, Jim Bogan reports at 
https://github.com/jbogan/dbdeploy/commit/563c7f02d86a4aab829c7f0e55a69d87dbbe1a
80 that this (understandably) caused formatting issues for views, packages etc.

dbdeploy should not needlessly trim provided statements.

Original issue reported on code.google.com by gtack...@googlemail.com on 8 Mar 2011 at 4:47

GoogleCodeExporter commented 9 years ago
Implemented in r151. Now trims whitespace at end of line only.

Original comment by gtack...@googlemail.com on 15 Mar 2011 at 4:25