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

List:       kde-commits
Subject:    Re: [calligra] libs: Fix find warnings
From:       Jaroslaw Staniek <staniek () kde ! org>
Date:       2015-09-01 0:32:55
Message-ID: CAOj7QQ2zwN-R=puL_L=Mo611heUEOs9VdkvXqBqAr-AJLBAD3w () mail ! gmail ! com
[Download RAW message or body]

Good catch, BTW there's no libs/db/ anymore in master so the exclusion can
be removed :)

On 31 August 2015 at 22:18, Albert Astals Cid <aacid@kde.org> wrote:

> Git commit 2c622e8a89d06b8be5235ccaaa89dde0c0200f12 by Albert Astals Cid.
> Committed on 31/08/2015 at 20:18.
> Pushed by aacid into branch 'master'.
>
> Fix find warnings
>
> find: invalid expression; you have used a binary operator '-o' with
> nothing before it.
>
> Please check it's what you really wanted
>
> CCMAIL: staniek@kde.org
>
> M  +1    -1    libs/Messages.sh
>
> http://commits.kde.org/calligra/2c622e8a89d06b8be5235ccaaa89dde0c0200f12
>
> diff --git a/libs/Messages.sh b/libs/Messages.sh
> index 5475304..fd8a7f1 100755
> --- a/libs/Messages.sh
> +++ b/libs/Messages.sh
> @@ -1,7 +1,7 @@
>  #! /bin/sh
>  source ../calligra_xgettext.sh
>
> -EXCLUDE="-o -path ./db/drivers/sqlite"
> +EXCLUDE="-path ./db/drivers/sqlite"
>  LIST=`find . \( $EXCLUDE \) -prune -o \( -name \*.ui -o -name \*.rc \)
> -type f -print | grep -v -e '/\.'`
>  $EXTRACTRC $LIST >> rc.cpp
>  LIST=`find . \( $EXCLUDE \) -prune -o \( -name \*.cpp -o -name \*.cc -o
> -name \*.h \) -type f -print | grep -v '/tests/' | grep -v -e '/\.'`
>



-- 
regards, Jaroslaw Staniek

KDE:
: A world-wide network of software engineers, artists, writers, translators
: and facilitators committed to Free Software development - http://kde.org
Calligra Suite:
: A graphic art and office suite - http://calligra.org
Kexi:
: A visual database apps builder - http://calligra.org/kexi
Qt Certified Specialist:
: http://www.linkedin.com/in/jstaniek

[Attachment #3 (text/html)]

<div dir="ltr"><div class="gmail_default" \
style="font-family:monospace,monospace;font-size:small">Good catch, BTW there&#39;s \
no libs/db/ anymore in master so the exclusion can be removed :)<br></div></div><div \
class="gmail_extra"><br><div class="gmail_quote">On 31 August 2015 at 22:18, Albert \
Astals Cid <span dir="ltr">&lt;<a href="mailto:aacid@kde.org" \
target="_blank">aacid@kde.org</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">Git commit 2c622e8a89d06b8be5235ccaaa89dde0c0200f12 by Albert \
Astals Cid.<br> Committed on 31/08/2015 at 20:18.<br>
Pushed by aacid into branch &#39;master&#39;.<br>
<br>
Fix find warnings<br>
<br>
find: invalid expression; you have used a binary operator &#39;-o&#39; with nothing \
before it.<br> <br>
Please check it&#39;s what you really wanted<br>
<br>
CCMAIL: <a href="mailto:staniek@kde.org">staniek@kde.org</a><br>
<br>
M   +1      -1      libs/Messages.sh<br>
<br>
<a href="http://commits.kde.org/calligra/2c622e8a89d06b8be5235ccaaa89dde0c0200f12" \
rel="noreferrer" target="_blank">http://commits.kde.org/calligra/2c622e8a89d06b8be5235ccaaa89dde0c0200f12</a><br>
 <br>
diff --git a/libs/Messages.sh b/libs/Messages.sh<br>
index 5475304..fd8a7f1 100755<br>
--- a/libs/Messages.sh<br>
+++ b/libs/Messages.sh<br>
@@ -1,7 +1,7 @@<br>
  #! /bin/sh<br>
  source ../calligra_xgettext.sh<br>
<br>
-EXCLUDE=&quot;-o -path ./db/drivers/sqlite&quot;<br>
+EXCLUDE=&quot;-path ./db/drivers/sqlite&quot;<br>
  LIST=`find . \( $EXCLUDE \) -prune -o \( -name \*.ui -o -name \*.rc \) -type f \
-print | grep -v -e &#39;/\.&#39;`<br>  $EXTRACTRC $LIST &gt;&gt; rc.cpp<br>
  LIST=`find . \( $EXCLUDE \) -prune -o \( -name \*.cpp -o -name \*.cc -o -name \*.h \
\) -type f -print | grep -v &#39;/tests/&#39; | grep -v -e &#39;/\.&#39;`<br> \
</blockquote></div><br><br clear="all"><br>-- <br><div \
class="gmail_signature">regards, Jaroslaw Staniek<br><br>KDE:<br>: A world-wide \
network of software engineers, artists, writers, translators<br>: and facilitators \
committed to Free Software development - <a href="http://kde.org" \
target="_blank">http://kde.org</a><br>Calligra Suite:<br>: A graphic art and office \
suite - <a href="http://calligra.org" \
target="_blank">http://calligra.org</a><br>Kexi:<br>: A visual database apps builder \
- <a href="http://calligra.org/kexi" \
target="_blank">http://calligra.org/kexi</a><br>Qt Certified Specialist:<br>: <a \
href="http://www.linkedin.com/in/jstaniek" \
target="_blank">http://www.linkedin.com/in/jstaniek</a></div> </div>



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