FW: Gideon alpha/patches

[GEHRMANN BERND]

-----Original Message-----
From: Dorin Lazar [mailto:lazar at deuromedia.ro]
Sent: lunedi, 23. giugno 2003 13:41
To: bernd at kdevelop.org
Subject: Gideon alpha/patches


    Hello,

    I managed to get my hands on a copy of the sources of gideon from
cvs, and 
started to play with it. I found two problems I crashed into the first
time I 
started it... I don't know if this is the right way to submit 
reports/patches, but maybe you would like to look at these two patches I

produced this weekend...
    the first problem is not actually a problem, but quite-an-annoying 
behaviour of the appwizard: if I input a project-name that is not allow
I 
will continue to setup my project details , and when I would press
'finish' 
it would get me back to the first page to correct the project name. So
one 
solution to not let this happen is to verify in-place the project name -
and 
this can be done by hooking a QValidator to the projectname_edit. The
first 
patch would do just that
    the second problem relates CVS and (C/C++, but I guess it works
likewise 
for all the project-types) initialization of the project (after creation
with 
the appwizard). If the user does not specify a CVSROOT gideon would
issue a 
command stating something like: cvs -d '' .... causing cvs to return an
err, 
and not completing initialisation. The second sub-task is the repository
name 
and this one is quite dangerous because if it's not specified it will
make 
the project import it's file into the root of the CVS server. The second

patch solves these problems by 1) removing the "-d '' " if necessary and
2) 
if no repository is specified, assign a repository with the name of the 
project.

    I don't know if this is the right way to get things done - however I
saw 
your name stated in both cvs and appwizard parts headers, so I
guessed... 
Well... thank you for your time, hope it wasn't too much trouble, and I
would 
apreciate if you could guide me to the right way of doing things like 
these...
   Yours, Dorin Lazar
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: CVSInteraction.desc
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20030623/a73a22d4/attachment.ksh>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ProjectNameValidation.patch
Type: text/x-diff
Size: 1645 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20030623/a73a22d4/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: CVSInteraction.patch
Type: text/x-diff
Size: 1804 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20030623/a73a22d4/attachment-0001.bin>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ProjectNameValidation.desc
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20030623/a73a22d4/attachment-0001.ksh>