Bruno Wolff III wrote: > On Sat, Aug 04, 2007 at 00:29:41 +0200, > Nils Kneuper <[EMAIL PROTECTED]> wrote: >> So in short: The server is now completely wiped clean and ATM no >> incompatible content is available on the dev campaign >> server (hehe, there is no content at all...). No idea how this did happen >> but I think it is not too bad since I >> eventually wanted to clean the server up anyway. I think a long search for >> the error is not this important either >> because Mordante wanted to start on rewriting the whole campaignserver stuff >> shortly. > > I think it would be a nice future addition to the campaign server to only > allow stuff that passed wmllint (or the equivalent) before being accepted > by the campaign server.
Quite frankly, I think this is a bad idea. I think it's a good idea to do a rudimentary syntax check on add-ons; basically the equivalent of running it through the WML preprocessor in order to check that it produces no preprocessor errors (unclosed tags, wrong number of macro arguments, that stuff), and that it doesn't otherwise crash Wesnoth when loaded. If wmllint (or equivalent) can do that or be made to do that, then great, that sounds good. Beyond that, I think there should not be further requirements. If someone has for example broken image references in his add-on, fine. No sense in blocking him from uploading it just for that, IMO. You can have broken references in a WIP scenario that isn't wired in to the campaign yet, you can have references that point to another add-on (some campaign depending on the imperial era, for example), and so on. However, it would probably be a good idea to display the results (and save them to a file, too) of wmllint (or whatever), if it spots anything suspicious, to the user after the upload just as a friendly reminder of things that are (possibly) broken in the add-on. -- Lari Nieminen [EMAIL PROTECTED] +358443758373 _______________________________________________ Wesnoth-dev mailing list [email protected] https://mail.gna.org/listinfo/wesnoth-dev
