[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

sql/modules/LOADORDER - redundant, remove?



1) sql/modules/README refers to the LOADORDER file
2) install-mycompany.sh uses sql/modules/install.sql

These define a slightly different order.
The order used in sql/modules/install.sql works correctly.
I haven't manually tested the order defined in LOADORDER.

There doesn't seem to be much point in maintaining duplicate information, especially when they don't agree.

Recommendation:
1) drop the LOADORDER file
2) change README to refer to install.sql as the definitive source instead.

To wit, in README, change:
The load order for the files is documented in sql/modules/LOADORDER

to:
The file install.sql loads the scripts in the correct order.  Other orderings
*may* work but are not supported.

and remove LOADORDER altogether.

--
Thank you,
-Adam Thompson
 <..hidden..>
 (204) 291-7950