[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #18 from Rex Dieter rdie...@math.unl.edu ---
New Package SCM Request
===
Package Name: qca
Short Description: Qt Cryptographic Architecture
Upstream URL:  http://delta.affinix.com/qca
Owners: rdieter slankes
Branches: f21 f20
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #17 from Rex Dieter rdie...@math.unl.edu ---
New Package SCM Request
===
Package Name: qca
Short Description: Qt Cryptographic Architecture
Upstream URL:  http://delta.affinix.com/qca
Owners: rdieter, slankes
Branches: f21,f20
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #19 from Rex Dieter rdie...@math.unl.edu ---
Forgot, the package exists, but is retired and just needs some new branches.

Package Change Request
==
Package Name: qca
New Branches: f21 f20
Owners: rdieter slankes

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #20 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2014-12-01 10:54:19



--- Comment #21 from Rex Dieter rdie...@math.unl.edu ---
Imported, thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

  Flags||needinfo?(dr...@land.ru)



--- Comment #22 from Rex Dieter rdie...@math.unl.edu ---
Boo, Qt5 build uses the same library soname and pkgconfig file names, so not
parallel-installable with Qt4 versions.

Ivan, would you accept a patch to rename those to include a -qt5 suffix (when
building against Qt5)?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #23 from Rex Dieter rdie...@math.unl.edu ---
And cmake files too.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|he...@kde.org



--- Comment #24 from Rex Dieter rdie...@math.unl.edu ---
Looks like I might be able to set QCA_SUFFIX=qt5 for this purpose, but I'd
argue this should be set automatically in a standard way, else we risk
API/ABI-incompatible implementations for different distros choosing different
values for QCA_SUFFIX.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #25 from Rex Dieter rdie...@math.unl.edu ---
cmake files didn't use QCA_SUFFIX (yet), here's the patch I'm using currently,
http://pkgs.fedoraproject.org/cgit/qca.git/tree/qca-2.1.0-cmake_QCA_SUFFIX.patch

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-12-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Ivan Romanov dr...@land.ru changed:

   What|Removed |Added

  Flags|needinfo?(dr...@land.ru)|



--- Comment #26 from Ivan Romanov dr...@land.ru ---
 but I'd argue this should be set automatically in a standard way

You are not alone :( https://git.reviewboard.kde.org/r/121168/

In really you must use two patches for Qt5 coinstalled version. 
http://quickgit.kde.org/?p=qca.gita=commith=9a9c16dfd1affc92962acd92e4a5246830ccbc04
http://quickgit.kde.org/?p=qca.gita=commith=66447d0454591f4c1deb5f4c988c6027194b1335

Anyway this all out of scope. Please make separate bugreport if you need/want.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Helio Chissini de Castro he...@kde.org changed:

   What|Removed |Added

 CC||he...@kde.org
  Flags||fedora-review+



--- Comment #16 from Helio Chissini de Castro he...@kde.org ---
Package is fine. About time to normalize names.

APPROVED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #11 from Ivan Romanov dr...@land.ru ---
$ rpm -qpl /home/taurus/rpmbuild/RPMS/x86_64/qca-2.1.0-2.fc20.R.x86_64.rpm
/usr/bin/mozcerts
/usr/bin/qcatool
/usr/lib64/libqca.so.2
/usr/lib64/libqca.so.2.1.0

It is bad idea to mix library and executable files. In this case I can't
install on my system both qca.x86_64 and qca.i686 pacakges. Also mozcerts and
qcatool not need for regular users. So this files should be moved to separate
subpackes qca-tools for example or something like.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #12 from Ivan Romanov dr...@land.ru ---
I wonder is it OK that package doesn't provide libqca-2.1.so ?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #13 from Rex Dieter rdie...@math.unl.edu ---
Re: comment #11
binaries should be ok, rpm uses binary coloring to prefer native binaries in
this context.

Not sure what you mean by comment #12 , the library soname is libqca.so.2 (not
libqca-2-1.so)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #14 from Ivan Romanov dr...@land.ru ---
[root@lix lib64]# ll libQtCore.so.*
lrwxrwxrwx. 1 root root  18 Nov 14 02:14 libQtCore.so.4 -
libQtCore.so.4.8.6
lrwxrwxrwx. 1 root root  18 Nov 14 02:14 libQtCore.so.4.8 -
libQtCore.so.4.8.6
-rwxr-xr-x  1 root root 3108864 Jul 24 00:32 libQtCore.so.4.8.6
[root@lix lib64]# ll libqca.so.*
lrwxrwxrwx. 1 root root  15 Nov 14 02:15 libqca.so.2 - libqca.so.2.0.3
lrwxrwxrwx. 1 root root  15 Nov 14 02:15 libqca.so.2.0 - libqca.so.2.0.3
-rwxr-xr-x  1 root root 1154456 Aug  4  2013 libqca.so.2.0.3

In new qca no libqca.so.2.1 . There are only libqca.so.2 and libqca.so.2.1.0.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #15 from Ivan Romanov dr...@land.ru ---
(In reply to Rex Dieter from comment #13)
 Re: comment #11
 binaries should be ok, rpm uses binary coloring to prefer native binaries in
 this context.
Yes. I checked it. No any problems. I installed both qca.i686 and qca.x86_64
# rpm -qa | grep qca
qca-devel-2.1.0-2.fc20.i686
qca-2.1.0-2.fc20.x86_64
qca-ossl-2.1.0-2.fc20.x86_64
qca-devel-2.1.0-2.fc20.x86_64
qca-gnupg-2.1.0-2.fc20.x86_64
qca-2.1.0-2.fc20.i686

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 CC||dr...@land.ru



--- Comment #2 from Rex Dieter rdie...@math.unl.edu ---
*** Bug 1161767 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

  Alias||qca



--- Comment #1 from Rex Dieter rdie...@math.unl.edu ---
This is a update and rename of existing qca2 package

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Blocks||512000




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=512000
[Bug 512000] rename qca2 - qca
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Blocks||928937 (qt-reviews)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=928937
[Bug 928937] Qt-related package review tracker
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #3 from Rex Dieter rdie...@math.unl.edu ---
Fyi, qca can now support Qt5 too, but I chose to not include that yet, in order
to keep the review simpler.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Ivan Romanov dr...@land.ru changed:

   What|Removed |Added

 Blocks|928937 (qt-reviews) |



--- Comment #4 from Ivan Romanov dr...@land.ru ---
Please add documentation and botan plugins subpackages.
Use make doc to build documentation. Seems I didn't do install target for docs.
So I must be copied to installation folder.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=928937
[Bug 928937] Qt-related package review tracker
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348

Ivan Romanov dr...@land.ru changed:

   What|Removed |Added

 Blocks||928937 (qt-reviews)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=928937
[Bug 928937] Qt-related package review tracker
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #5 from Rex Dieter rdie...@math.unl.edu ---
Spec URL: https://rdieter.fedorapeople.org/rpms/qca/qca.spec
SRPM URL: https://rdieter.fedorapeople.org/rpms/qca/qca-2.1.0-2.fc20.src.rpm

%changelog
* Fri Nov 14 2014 Rex Dieter rdie...@fedoraproject.org 2.1.0-2
- -botan, -doc subpkgs, and READMEs to plugin subpkgs

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #6 from Ivan Romanov dr...@land.ru ---
#The following tests FAILED:
#  8 - FileWatch (Failed) 
How I can reproduce this?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #7 from Ivan Romanov dr...@land.ru ---
Maybe should use %{_docdir} instead of %{_qt4_docdir}? Or there are reasons to
use %{_qt4_docdir}?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #8 from Rex Dieter rdie...@math.unl.edu ---
Considering we will likely be adding qt5 support in the near future, using a Qt
agnostic dir makes sense.  I'll switch that in the next pkg iteration (but
won't update the review *just* for that.

As far as the failed test, good question, maybe the mock/koji environment is
different somehow.  You should be able to reproduce it in mock yourself, then
run 'mock shell' to entire into the buildroot for more investigation if needed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #9 from Ivan Romanov dr...@land.ru ---
Need to gnupg to BR. It is used in tests.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1164348] Review Request: qca - Qt Cryptographic Architecture

2014-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164348



--- Comment #10 from Ivan Romanov dr...@land.ru ---
I rebuild qca for F21 x68_64 in mock. But can't reproduce FileWatcher failing
unittest

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review