[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-09-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #48 from Fedora Update System upda...@fedoraproject.org ---
owncloud-csync-0.50.8-9.fc18, mirall-1.0.5-4.fc18 has been pushed to the Fedora
18 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2012-08-24 15:22:59

--- Comment #47 from Fedora Update System upda...@fedoraproject.org ---
owncloud-csync-0.50.8-9.fc17, mirall-1.0.5-4.fc17 has been pushed to the Fedora
17 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #45 from Fedora Update System upda...@fedoraproject.org ---
owncloud-csync-0.50.8-9.fc17,mirall-1.0.5-4.fc17 has been submitted as an
update for Fedora 17.
https://admin.fedoraproject.org/updates/owncloud-csync-0.50.8-9.fc17,mirall-1.0.5-4.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #46 from Fedora Update System upda...@fedoraproject.org ---
owncloud-csync-0.50.8-9.fc18,mirall-1.0.5-4.fc18 has been submitted as an
update for Fedora 18.
https://admin.fedoraproject.org/updates/owncloud-csync-0.50.8-9.fc18,mirall-1.0.5-4.fc18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #42 from Fedora Update System upda...@fedoraproject.org ---
owncloud-csync-0.50.8-8.fc17,mirall-1.0.5-3.fc17 has been submitted as an
update for Fedora 17.
https://admin.fedoraproject.org/updates/owncloud-csync-0.50.8-8.fc17,mirall-1.0.5-3.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #43 from Fedora Update System upda...@fedoraproject.org ---
owncloud-csync-0.50.8-8.fc18,mirall-1.0.5-3.fc18 has been submitted as an
update for Fedora 18.
https://admin.fedoraproject.org/updates/owncloud-csync-0.50.8-8.fc18,mirall-1.0.5-3.fc18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #44 from Fedora Update System upda...@fedoraproject.org ---
owncloud-csync-0.50.8-8.fc17, mirall-1.0.5-3.fc17 has been pushed to the Fedora
17 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #30 from Gregor Tätzner gre...@freenet.de ---
looks good, except that the file names still indicate a unmodified csync. If
Andreas is ever going to package stock csync, this will generate file
conflicts.

After all comment #20 from rdieter is still valid. You can try to patch the
CMakeLists.txt files to change the names of the generated libs.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #31 from Joseph Marrero jmarr...@gmail.com ---
I made 6 patches for the CMakeLists in the package, it builds and installs ok.

NEW SRPM:
http://jmarrero.fedorapeople.org/packages/owncloud-csync/owncloud-csync-0.50.8-7.fc17.src.rpm
NEW SPEC:
http://jmarrero.fedorapeople.org/packages/owncloud-csync/owncloud-csync.spec
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4401620


All old files are in:
http://jmarrero.fedorapeople.org/packages/owncloud-csync/old/*

Please let me know if there is anything else.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #32 from Joseph Marrero jmarr...@gmail.com ---
Also I made this:
https://docs.google.com/folder/d/0B0IbyJGzQiLLRXBiZVRkMTk4bWc/edit

http://koji.fedoraproject.org/koji/taskinfo?taskID=4401292

This Is mirall with csync integrated as described here:
http://owncloud.org/dev/sync-clients/linux-build/

The thing with this is that it builds just fine in Koji. But when Installing
mirall-common installs just fine but when installing owncloud-client or mirall
then I get this:

Error: Package: mirall-1.0.5-3.fc17.x86_64 (/mirall-1.0.5-3.fc17.x86_64)
   Requires: libcsync.so.0()(64bit)
 You could try using --skip-broken to work around the problem
 You could try running: rpm -Va --nofiles --nodigest



I checked the spec there is not csync dep there and while building CMAKE find
csync libs and builds mirall and owncloud-client just fine.

But when Installing it complains about libcsync, in the above link It says that
if I build mirall this way there is no need to install csync on the machine.

Let me know what path to pursue
and what goes better along the guidelines of fedora.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #33 from Rex Dieter rdie...@math.unl.edu ---
Since we're changing the library soname, you'll need to rebuild mirall against
the new one (else, it still wants the old name, as you discoverred).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #34 from Gregor Tätzner gre...@freenet.de ---
(In reply to comment #32)
 Also I made this:
 https://docs.google.com/folder/d/0B0IbyJGzQiLLRXBiZVRkMTk4bWc/edit
 
 http://koji.fedoraproject.org/koji/taskinfo?taskID=4401292
 
 This Is mirall with csync integrated as described here:
 http://owncloud.org/dev/sync-clients/linux-build/

Actually this is not about integrating csync into mirall (in other terms:
linking the csync lib statically). That page just describes how you can build
mirall without having to install the csync devel files system wide.
Nevertheless you would need libcsync at runtime in libdir. And thats your
problem here: rpm detects the runtime deps of mirall but can't find libcsync
(and libocsync is not suitable because you build mirall against libcsync)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #35 from Gregor Tätzner gre...@freenet.de ---
apart from that, please mark the doc packages as NoArch. just html stuff in
there

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #36 from Joseph Marrero jmarr...@gmail.com ---
Ok,
I will continue the path of creating the owncloud-csync and mirall packages
separate, doing the last build of owncloud-csync to mark NoArch the docs and
moving to mirall.

Let me know what else is needed to pass the review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

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

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|rdie...@math.unl.edu
  Flags||fedora-review?

--- Comment #37 from Rex Dieter rdie...@math.unl.edu ---
Ok, I'll do the formal review, since no one's assigned the bug or set the
review flag yet.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #38 from Joseph Marrero jmarr...@gmail.com ---
Added NoArch to the doc packages builded all OK :)

NEW SRPM:
http://jmarrero.fedorapeople.org/packages/owncloud-csync/owncloud-csync-0.50.8-8.fc17.src.rpm
NEW SPEC:
http://jmarrero.fedorapeople.org/packages/owncloud-csync/owncloud-csync.spec

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4401867

All old files are in:
http://jmarrero.fedorapeople.org/packages/owncloud-csync/old/*

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

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

   What|Removed |Added

 Status|NEW |ASSIGNED
  Flags|fedora-review?  |fedora-review+

--- Comment #39 from Rex Dieter rdie...@math.unl.edu ---
naming: ok

sources: ok
$ md5sum *.bz2
322f6fa22ca0e8cd05f23dc0d075e7ca  csync-0.50.8.tar.bz2

licensing: ok

macros: ok

scriptlets: ok

builds/installs: ok

thanks, looks good now.


APPROVED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

Joseph Marrero jmarr...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
  Flags|fedora-review+  |fedora-review?, fedora-cvs?

--- Comment #40 from Joseph Marrero jmarr...@gmail.com ---
New Package SCM Request
===
Package Name: owncloud-csync
Short Description: a file synchroniser utility
Owners: jmarrero
Branches: f17 f18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

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

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #41 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

Kevin Kofler ke...@tigcc.ticalc.org changed:

   What|Removed |Added

Summary|Review Request: csync - a   |Review Request:
   |file synchroniser utility   |owncloud-csync - a file
   ||synchroniser utility
  Alias|csync   |owncloud-csync

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #28 from Kevin Kofler ke...@tigcc.ticalc.org ---
Andreas, if you still want to get vanilla csync in, we can reopen bug #844960
as long as nobody else submits it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848208] Review Request: owncloud-csync - a file synchroniser utility

2012-08-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848208

--- Comment #29 from Kevin Kofler ke...@tigcc.ticalc.org ---
Sorry, wrong bug ID…

Andreas, if you still want to get vanilla csync in, we can reopen bug #565902
as long as nobody else submits it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review