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

List:       kde-bindings
Subject:    Re: [Kde-bindings] [Qyoto] New Build Instructions for Linux, Windows MinGW and Windows Visual Studio
From:       Dimitar Dobrev <dpldobrev () yahoo ! com>
Date:       2013-03-04 10:52:09
Message-ID: 1362394329.30318.YahooMailNeo () web122402 ! mail ! ne1 ! yahoo ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


    

    Unfortunately, that did not work. The problem was that these KDE policies do not \
check just HEAD but all commits in line so the offending commit still caused trouble. \
I found which one it was: "get rid of compiler warnings about mstackrealign in visual \
studio". If you look at your diff for this commit, you'll see that every file had \
only the region around "# Realign the stack..." changed - with the exception of \
qyoto-phonon/native/CMakeLists.txt, which had almost all of its new lines changed. I \
had to manually reapply your changes from this commit and above and create new \
commits to make this work. This means that you may have some conflicts to resolve \
when you merge to your dev branch.  Anyway, while this was not pleasant, I really \
appreciate your work, please keep up. :)

    Regards,
    Dimitar



________________________________
 From: David Lechner <david@lechnology.com>
To: kde-bindings@kde.org 
Sent: Sunday, March 3, 2013 11:14 PM
Subject: Re: [Kde-bindings] [Qyoto] New Build Instructions for Linux, Windows MinGW \
and Windows Visual Studio  
On 3/3/2013 9:15 AM, Dimitar Dobrev wrote:
> 
> This works, thanks. However, now I get an error trying to push the
> last commit:
> 
> remote: Audit failure - Commit 8e56f67f29dee0c30e620d7d1c55faac8c06723e
> - This file contains wrong line endings, it needs to be linux line
> endings ( you can look at dos2unix if needed ) :
> assemblies/qyoto-phonon/native/CMakeLists.txt
> remote: Push declined - commits failed audit
> remote: error: hook declined to update refs/heads/master
> To git@git.kde.org:assemblygen
> ! [remote rejected] master -> master (hook declined)
> error: failed to push some refs to 'git@git.kde.org:assemblygen'
> 
> I push from Windows and I use Git's automatic conversion from LF to
> CRLF. So qyoto-phonon/native/CMakeLists.txt should've appeared to me
> with CRLF but when I open it I can see it uses LF. All other
> native/CMakeLists.txt appear with CRLF. Could you please check this as well?

I think you make have said this backward. The error message says that everything \
should be linux line endings (LF). Looking at all my files, they are LF only - except \
for one line in assemblies/qyoto-phonon/native/CMakeLists.txt, which I have fixed: \
https://github.com/dlech/assemblygen/commit/ff02487b4aedf38c95e0246f608f98cf773fda7b

I'll be sure to change the settings on my local repo to keep this from happening in \
the future.

So, cross your fingers. Hope we got it this time :)


_______________________________________________
Kde-bindings mailing list
Kde-bindings@kde.org
https://mail.kde.org/mailman/listinfo/kde-bindings


[Attachment #5 (text/html)]

<html><body><div style="color:#000; background-color:#fff; font-family:times new \
roman, new york, times, serif;font-size:12pt"><span class="tab">&nbsp;&nbsp;&nbsp; \
<br></span><div style="color: rgb(0, 0, 0); font-size: 16px; font-family: times new \
roman,new york,times,serif; background-color: transparent; font-style: normal;"><span \
class="tab">&nbsp;&nbsp;&nbsp; Unfortunately, that did not work. The problem was that \
</span>these KDE policies do not check just HEAD but all commits in line so the \
offending commit still caused trouble. I found which one it was: "get rid of compiler \
warnings about mstackrealign in visual studio". If you look at your diff for this \
commit, you'll see that every file had only the region around "# Realign the \
stack..." changed - with the exception of qyoto-phonon/native/CMakeLists.txt, which \
had almost all of its new lines changed. I had to manually reapply your changes from \
this commit and above and create new commits to make  this work. This means that you \
may have some conflicts to resolve when you merge to your dev branch.<br><span \
class="tab">&nbsp;&nbsp;&nbsp; Anyway, while this was not pleasant, I really \
appreciate your work, please keep up. :)</span></div><div style="color: rgb(0, 0, 0); \
font-size: 16px; font-family: times new roman,new york,times,serif; background-color: \
transparent; font-style: normal;"><br><span class="tab"></span></div><div \
style="color: rgb(0, 0, 0); font-size: 16px; font-family: times new roman,new \
york,times,serif; background-color: transparent; font-style: normal;"><span \
class="tab">&nbsp;&nbsp;&nbsp; Regards,</span></div><div style="color: rgb(0, 0, 0); \
font-size: 16px; font-family: times new roman,new york,times,serif; background-color: \
transparent; font-style: normal;"><span class="tab">&nbsp;&nbsp;&nbsp; \
Dimitar</span><span class="tab"><br></span></div><div><br></div>  <div \
style="font-family: times new roman, new york, times, serif;  font-size: 12pt;"> <div \
style="font-family: times new roman, new york, times, serif; font-size: 12pt;"> <div \
dir="ltr"> <font face="Arial" size="2"> <hr size="1">  <b><span \
style="font-weight:bold;">From:</span></b> David Lechner \
&lt;david@lechnology.com&gt;<br> <b><span style="font-weight: bold;">To:</span></b> \
kde-bindings@kde.org <br> <b><span style="font-weight: bold;">Sent:</span></b> \
Sunday, March 3, 2013 11:14 PM<br> <b><span style="font-weight: \
bold;">Subject:</span></b> Re: [Kde-bindings] [Qyoto] New Build Instructions for \
Linux, Windows MinGW and Windows Visual Studio<br> </font> </div> <br> On 3/3/2013 \
9:15 AM, Dimitar Dobrev wrote:<br>&gt; <br>&gt;&nbsp; &nbsp; &nbsp; This works, \
thanks. However, now I get an error trying to push the<br>&gt; last commit:<br>&gt; \
<br>&gt; remote: Audit failure - Commit \
8e56f67f29dee0c30e620d7d1c55faac8c06723e<br>&gt; - This file contains wrong line \
endings, it needs to be linux line<br>&gt; endings ( you can look at dos2unix if \
needed ) :<br>&gt; assemblies/qyoto-phonon/native/CMakeLists.txt<br>&gt; remote: Push \
declined - commits failed audit<br>&gt; remote: error: hook declined to update \
refs/heads/master<br>&gt; To <a ymailto="mailto:git@git.kde.org" \
href="mailto:git@git.kde.org">git@git.kde.org</a>:assemblygen<br>&gt; ! [remote \
rejected] master -&gt; master (hook declined)<br>&gt; error: failed to push some refs \
to '<a ymailto="mailto:git@git.kde.org" \
href="mailto:git@git.kde.org">git@git.kde.org</a>:assemblygen'<br>&gt; <br>&gt;&nbsp; \
&nbsp; &nbsp; I push from Windows and I use Git's automatic  conversion from LF \
to<br>&gt; CRLF. So qyoto-phonon/native/CMakeLists.txt should've appeared to \
me<br>&gt; with CRLF but when I open it I can see it uses LF. All other<br>&gt; \
native/CMakeLists.txt appear with CRLF. Could you please check this as well?<br><br>I \
think you make have said this backward. The error message says that everything should \
be linux line endings (LF). Looking at all my files, they are LF only - except for \
one line in assemblies/qyoto-phonon/native/CMakeLists.txt, which I have fixed: <a \
href="https://github.com/dlech/assemblygen/commit/ff02487b4aedf38c95e0246f608f98cf773fda7b" \
target="_blank">https://github.com/dlech/assemblygen/commit/ff02487b4aedf38c95e0246f608f98cf773fda7b</a><br><br>I'll \
be sure to change the settings on my local repo to keep this from happening in the \
future.<br><br>So, cross your fingers. Hope we got it this time \
:)<br><br><br>_______________________________________________<br>Kde-bindings mailing \
list<br><a  ymailto="mailto:Kde-bindings@kde.org" \
href="mailto:Kde-bindings@kde.org">Kde-bindings@kde.org</a><br><a \
href="https://mail.kde.org/mailman/listinfo/kde-bindings" \
target="_blank">https://mail.kde.org/mailman/listinfo/kde-bindings</a><br><br><br> \
</div> </div>  </div></body></html>



_______________________________________________
Kde-bindings mailing list
Kde-bindings@kde.org
https://mail.kde.org/mailman/listinfo/kde-bindings


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

Configure | About | News | Add a list | Sponsored by KoreLogic