'Re: [PATCH] Support for in-source builds in kdesvn-build'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: [PATCH] Support for in-source builds in kdesvn-build
From:       Michael Pyne <mpyne () purinchu ! net>
Date:       2008-09-28 2:15:36
Message-ID: 200809272215.47089.mpyne () purinchu ! net
[Download RAW message or body]

[Attachment #2 (multipart/signed)]

[Attachment #4 (multipart/alternative)]

On Friday 26 September 2008, Robert Zwerus wrote:
> Hey people
>
> I'd like to use kdesvn-build and Eclipse together. Unfortunately, Eclipse
> makes working with out-of-source builds a bit harder than necessary, so I
> opt for in-source builds.
>
> I added support for it to the kdesvn-build script, a patch against the
> current SVN version is attached.
>
> When it's okay with you, I'd like to commit it.

Robert,

I see you emailed me about the patch, I apologize for not having time to look 
at it.

But first I'd like to ask you to test the following setting for the build-dir 
option:

  build-dir ${source-dir}

This should set the build-dir to whatever the source-dir is for that module 
(even if you only make this change in your global options).  My testing seems 
to show that this will only work if you use a source-dir of ~/foo/bar or 
/foo/bar (and not simply foo/bar).

I'd like to avoid patching if possible, especially if this solution works 
(especially since I don't think it seems to much of a hack, situations like 
this is why I added support for ${option-name} in the config file)

Regards,
 - Michael Pyne

[Attachment #7 (text/html)]

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" \
"http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" \
content="1" /><style type="text/css">p, li { white-space: pre-wrap; \
}</style></head><body style=" font-family:'Consolas'; font-size:11pt; \
font-weight:400; font-style:normal;">On Friday 26 September 2008, Robert Zwerus \
wrote:<br> &gt; Hey people<br>
&gt;<br>
&gt; I'd like to use kdesvn-build and Eclipse together. Unfortunately, Eclipse<br>
&gt; makes working with out-of-source builds a bit harder than necessary, so I<br>
&gt; opt for in-source builds.<br>
&gt;<br>
&gt; I added support for it to the kdesvn-build script, a patch against the<br>
&gt; current SVN version is attached.<br>
&gt;<br>
&gt; When it's okay with you, I'd like to commit it.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; \
margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; \
-qt-user-state:0;"><br></p>Robert,<br> <p style="-qt-paragraph-type:empty; \
margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; \
-qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>I see you emailed me \
about the patch, I apologize for not having time to look at it.<br> <p \
style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; \
margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>But \
first I'd like to ask you to test the following setting for the build-dir option:<br> \
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; \
margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; \
-qt-user-state:0;"><br></p>  build-dir ${source-dir}<br> <p \
style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; \
margin-right:0px; -qt-block-indent:0; text-indent:0px; \
-qt-user-state:0;"><br></p>This should set the build-dir to whatever the source-dir \
is for that module (even if you only make this change in your global options).  My \
testing seems to show that this will only work if you use a source-dir of ~/foo/bar \
or /foo/bar (and not simply foo/bar).<br> <p style="-qt-paragraph-type:empty; \
margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; \
-qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>I'd like to avoid \
patching if possible, especially if this solution works (especially since I don't \
think it seems to much of a hack, situations like this is why I added support for \
${option-name} in the config file)<br> <p style="-qt-paragraph-type:empty; \
margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; \
                -qt-block-indent:0; text-indent:0px; \
                -qt-user-state:0;"><br></p>Regards,<br>
 - Michael Pyne</p></body></html>

["signature.asc" (application/pgp-signature)]

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

[prev in list] [next in list] [prev in thread] [next in thread] 