optional git patch Merc_Spells.sql fails about server HOT 4 CLOSED

Run the one in the svn folder first. Made the git folder specifically for only incremental updates on the assumption people applied all svn updates already. So if you don't have the tables already it wont work.

from server.

Thanks @KimLS. I followed the instructions very carefully and eventually got it working (I can log in and run around). I ended up not applying that optional patch that was giving me grief.

The real problem I have is there is a disconnect between the Windows Server Setup Guide, which was supposedly just recently updated to include this new conversion to git, and the code hosted here. It never specifically asks to source the bots.sql, botsconvert.sql, or mercs.sql and since they do not have numbers preceding them, it's difficult for someone to tell if they are important or not. Maybe this is something you guys are figuring out as the conversion from svn to git moves forward, but I thought I would at least bring it to your attention.

Thanks again

from server.

That was a poor choice on my part. Mercs.sql was supposed to allow changes to be made to merc stats and spells lists frequently while mercs are being developed. This was supposed to allow you to source it once after the other merc sql files were run, and to not require those wanting to use mercs to have to source in bunches of updates ( 2 so far and another on the way already). The file should probably be renamed to 2380_optional_merc_stats_and_spells.sql.

A good source of information is the changelog, which can answer a lot of questions like these. All sql updates should be listed there, and can be used to match the source changes to the sql changes.

Re: bots- they are completely optional, and actually require a compile option to be made when using cmake, so a decision would have to be made whether to use bots or not. Not sourcing those files wouldn't impact anything in that case. Bot_convert.sql was used a long time ago when bots were stored in the database differently and would only be necessary to upgrade bots from a very old version ( and again should be in the changelog somewhere). Most people use the load_bots.sql from the peq database, but since PEQ doezn't use bots, it may or may not include all updated found in the sql/svn folder, but should be mentioned in the changelog.

Hope that helps.

from server.

Thanks guys.

from server.