Bug 187841 - kdesvn-build could warn about invalid module configuration files
Summary: kdesvn-build could warn about invalid module configuration files
Status: RESOLVED FIXED
Alias: None
Product: kdesrc-build
Classification: Developer tools
Component: general (show other bugs)
Version: Git
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: Michael Pyne
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-22 11:12 UTC by Andreas Pakulat
Modified: 2010-01-31 06:30 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Pakulat 2009-03-22 11:12:30 UTC
I just added this snippet to my .kdesvn-buildrc:
module palapeli
use-cmake true
cmake-options -DCMAKE_BUILD_TYPE=RelWithDebInfo
module-base-path trunk/playground/games/palapeli
end module

However when running kdesvn-build palapeli it searches for the new module in trunk/KDE and not in playground.
Comment 1 Andreas Pakulat 2009-03-22 11:14:49 UTC
Manually checking out palapeli into the source dir and then running kdesvn-build also doesn't work. 

kdesvn-build is r942628
Comment 2 Andreas Pakulat 2009-03-22 11:16:18 UTC
Hmm, also upon the very first invocation I got the following output, which vanished on the second try but came back right after removing the palapeli entry from .kdesvn-build-data:

Script started processing at Sun Mar 22 11:15:16 2009
<<<  Build Process  >>>
 *
 * Unknown module palapeli, configure it in /home/andreas/.kdesvn-buildrc.
 *

Building palapeli (1/1)
        Waiting for source code update.
svn: URL 'https://svn.kde.org/home/kde/trunk/KDE/palapeli' doesn't exist
Error updating palapeli, removing from list of packages to build.
        Unable to update palapeli, build canceled.

<<<  Build Done  >>>
Comment 3 Andreas Pakulat 2009-03-22 11:19:04 UTC
Hmm, interesting. Apparently the module _before_ the palapeli entry was broken, it used "end-module" instead of "end module" which caused a hickup for kdesvn-build.

Would be nice if kdesvn-build could warn about such errors instead of users having to guess why building a particular module doesn't work.
Comment 4 Michael Pyne 2010-01-31 06:26:54 UTC
SVN commit 1082663 by mpyne:

Add a sanity check to the rc file processing code to make sure new module sections are
complained about if they are wrong.

BUG:187841


 M  +8 -0      kdesvn-build  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1082663