[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Jonathan Underwood jonathan.underw...@gmail.com changed: What|Removed |Added Blocks||1243530 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1243530 [Bug 1243530] Review Request: winswitch - A tool which allows you to display running applications on other computers -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Jonathan Underwood jonathan.underw...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |WONTFIX Last Closed||2015-03-03 14:18:42 --- Comment #44 from Jonathan Underwood jonathan.underw...@gmail.com --- OK, I have updated the package and cloned this bug in order to take over the package submission process - see BZ#1198312. Reviewers very welcome. If tchollingsworth returns and wants to pick the package up again that's totally fine, but in the meantime this bug probably needs closing. -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Jonathan Underwood jonathan.underw...@gmail.com changed: What|Removed |Added Blocks||1198312 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1198312 [Bug 1198312] Review Request: xpra - screen for X -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #43 from Karel Volný kvo...@redhat.com --- (In reply to Antoine Martin from comment #41) I'm still puzzled about the fact that this ticket has been stuck for well over a year. seems everyone doing some work on this has run out of energy or time for this ... What's the procedure for getting this unstuck? if the original reporter won't answer within some reasonable timeframe (you can also consider trying to reach him via different channels), you can take over this review request meanwhile, you can propose a patch for the latest spec (comment #34) to update it to the new upstream version then a reviewer is needed ... you can take this role (or find someone else if you take over the request) please see https://fedoraproject.org/wiki/Package_Review_Process -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Jonathan Underwood jonathan.underw...@gmail.com changed: What|Removed |Added CC||jonathan.underw...@gmail.co ||m, ||tchollingswo...@gmail.com Flags||needinfo?(tchollingsworth@g ||mail.com) --- Comment #42 from Jonathan Underwood jonathan.underw...@gmail.com --- @tchollingsworth: are you still proposing this package for review? -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #41 from Antoine Martin anto...@nagafix.co.uk --- 0.14.0 is out and will be a LTS release (18 months+) tailor made for packaging into distributions. *Many* of the changes were in fact related to RPM packaging and security. In no small part trying to deal with the rather disappointing update schedule of media libraries which aren't part of Fedora proper. This may be OK for some when used through a media player only (I think not, but that may just be me), a lot less so when used with a network facing tool like xpra. Anyway, all this and more is discussed in greater detail here: http://xpra.org/trac/wiki/News#a0.14.0Release I'm still puzzled about the fact that this ticket has been stuck for well over a year. Especially considering that there are a number of Fedora users willing to help move things forward (myself included). What's the procedure for getting this unstuck? -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Nicolas Chauvet (kwizart) kwiz...@gmail.com changed: What|Removed |Added Status|ASSIGNED|NEW Assignee|kwiz...@gmail.com |nob...@fedoraproject.org Flags|fedora-review? | --- Comment #40 from Nicolas Chauvet (kwizart) kwiz...@gmail.com --- Ok, sorry but I was not the right person for review swap. -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Stephen Gauthier sgauth...@spikes.com changed: What|Removed |Added CC||sgauth...@spikes.com --- Comment #39 from Stephen Gauthier sgauth...@spikes.com --- I would also like to inquire as to the status of this review. Xpra is currently at stable 0.12.5 so this also probably needs some update. I'd be willing to contribute if there is something I could do to help. -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #38 from Antoine Martin anto...@nagafix.co.uk --- Oh, and since we are the new effective upstream for python-webm, it is worth mentioning that we have now fixed the memory leak in there too. AFAICT, this affects the version currently shipped in the repos for all Fedora releases. See here for details: http://xpra.org/trac/ticket/491#comment:4 python-pillow also leaks memory, but we're not fixing that one, the bug has been reported upstream instead (see link above - a test case is included). -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #37 from Antoine Martin anto...@nagafix.co.uk --- And now 0.12.0 is out: http://xpra.org/trac/wiki/News#a2014-03-23 Just one packaging update in there: libfakeXinerama has been added as an optional dependency for making fullscreen applications work with the dummy X11 server. Are there any hold ups? Anything I can help with? (for many common workloads, xpra is much more efficient than the alternatives currently packaged in the repos, it's a bit sad to see Fedora people stuck using early 1990s technologies) -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #36 from Antoine Martin anto...@nagafix.co.uk --- Please note that as of version 0.11.0, released just now: http://lists.devloop.org.uk/pipermail/shifter-users/2014-January/000792.html A number of dependencies have been added or changed - which may require changes to the spec file, OTOH: * python-lz4 is nice to have (much faster than zlib) and easy to package * we have a Cython colourspace conversion fallback, so vpx support no longer requires swscale (ffmpeg/libav) * we can take advantage of OpenCL or CUDA if present for colourspace conversion too (let's hope the OpenCL packaging gets sorted out) * NVENC (xpra exclusive) and CUDA are unlikely to be packaged by Fedora, but maybe for RHEL? * webp: found a memory leak in it, so this encoding is no longer used automatically (for lossless refresh/small regions) and very strongly discouraged - until I find the source of the leak. So the dependency can be dropped, at least for now. -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Sebastian Dyroff fed...@dyroff.org changed: What|Removed |Added CC||fed...@dyroff.org --- Comment #35 from Sebastian Dyroff fed...@dyroff.org --- Hey, what is the status of this package review? Are there some obvious problems which prevents inclusion into fedora? -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Nicolas Chauvet (kwizart) kwiz...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||kwiz...@gmail.com Assignee|nob...@fedoraproject.org|kwiz...@gmail.com Flags||fedora-review? --- Comment #33 from Nicolas Chauvet (kwizart) kwiz...@gmail.com --- - starting 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #34 from T.C. Hollingsworth tchollingswo...@gmail.com --- Sorry, meant to update this to the latest upstream last night but I forgot. :-( -- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.10.6-1.fc19.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6077461 * Fri Oct 18 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.10.6-1 - new upstream release 0.10.6 http://lists.devloop.org.uk/pipermail/shifter-users/2013-October/000726.html -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #29 from T.C. Hollingsworth tchollingswo...@gmail.com --- (In reply to Antoine Martin from comment #28) don't ship webm stuff that doesn't work without ffmpeg anyway I don't know who told you that, this is wrong. webm (aka webp) has nothing to do with ffmpeg. VPX, which is a distant cousin of webp, does require ffmpeg (for colourspace conversion via ffmpeg's swscale), this is a soft runtime dependency. FYI: it should even be possible to enable client VPX support without swscale installed when rendering to accelerated GL windows - this isn't implemented yet. I thought webm == VP8 and wouldn't work without swscale. If having the webm module around does do something even with vpx disabled, or if having libvpx enabled even with ffmpeg disabled is useful (or will be so in the future), I'd be happy to add them back. -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #30 from Antoine Martin anto...@nagafix.co.uk --- Although they are related, webm != vpx webm (aka webp) is for single frames, vpx is for video streams. Xpra's webp codec does not require anything beyond python-webm, be aware though that the upstream project is unresponsive and that others (at least Debian for sure) are now using our more up to date version. (which you will need to support webp encoding of transparent windows) Xpra's vpx codec requires swscale on the server for converting BGRA pixels from the X11 server into a YCbCr 420 planar. On the client side, vpx *could* work without swscale if we added code to enable vpx with the OpenGL codepath. I hope this clarifies things. -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #31 from T.C. Hollingsworth tchollingswo...@gmail.com --- (In reply to Antoine Martin from comment #30) Although they are related, webm != vpx webm (aka webp) is for single frames, vpx is for video streams. Ah, it does WebP not VP8. (And I should have remembered that since it uses libwebp and not libvpx, d'oh!) It turns out it wasn't showing up in xpra_launcher with my package because of an unbundling fail. :-/ Xpra's webp codec does not require anything beyond python-webm, be aware though that the upstream project is unresponsive and that others (at least Debian for sure) are now using our more up to date version. (which you will need to support webp encoding of transparent windows) I'm applying your patches to our separate python-webm too, BTW. :-) Xpra's vpx codec requires swscale on the server for converting BGRA pixels from the X11 server into a YCbCr 420 planar. On the client side, vpx *could* work without swscale if we added code to enable vpx with the OpenGL codepath. Okay, if client-side libvpx-without-swscale becomes a reality one day, please let me know and I'll enable it. In the meantime, I'd prefer not to drag in a libvpx dependency that won't do anything with the Fedora build (and might mislead people into thinking this hobbled build supports VP8 when it doesn't at all). People who want the video-based codecs instead of the image ones (i.e. everybody) will just have to grab them from that other repo. ;-) I hope this clarifies things. Thanks a lot! -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #32 from T.C. Hollingsworth tchollingswo...@gmail.com --- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.10.4-2.fc19.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6037266 * Tue Oct 08 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.10.4-2 - reenable webp support - fix webm unbundling to support importing all modules in the webm package - require latest python-webm so it matches what's bundled upstream -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #27 from T.C. Hollingsworth tchollingswo...@gmail.com --- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.10.4-1.fc19.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6033594 * Mon Oct 07 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.10.4-1 - rebase to 0.10.4 - don't ship webm stuff that doesn't work without ffmpeg anyway -- 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #28 from Antoine Martin anto...@nagafix.co.uk --- don't ship webm stuff that doesn't work without ffmpeg anyway I don't know who told you that, this is wrong. webm (aka webp) has nothing to do with ffmpeg. VPX, which is a distant cousin of webp, does require ffmpeg (for colourspace conversion via ffmpeg's swscale), this is a soft runtime dependency. FYI: it should even be possible to enable client VPX support without swscale installed when rendering to accelerated GL windows - this isn't implemented 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 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #26 from Antoine Martin anto...@nagafix.co.uk --- 0.10 released today, please be aware of some packaging issues that *will* affect the way you were planning on doing things (unbundling libs): http://lists.devloop.org.uk/pipermail/shifter-users/2013-August/000642.html -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=w4VSqikWwha=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #22 from T.C. Hollingsworth tchollingswo...@gmail.com --- Christopher: in comment 16 Karel said he wouldn't mind if someone takes this over if he gets busy and forgets about this, so feel free to take this over if you want this to get in faster. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=PrZu3CrzwAa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Assignee|kvo...@redhat.com |nob...@fedoraproject.org Flags|needinfo?(kvo...@redhat.com | |) | --- Comment #23 from Christopher Meng cicku...@gmail.com --- I'm sorry for some reason I can't take this. I'll leave it to nobody if you agree. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=WzcVDYtcKIa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Status|ASSIGNED|NEW -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ZnlMMde6jsa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 T.C. Hollingsworth tchollingswo...@gmail.com changed: What|Removed |Added Flags|fedora-review? | --- Comment #24 from T.C. Hollingsworth tchollingswo...@gmail.com --- (In reply to Christopher Meng from comment #23) I'm sorry for some reason I can't take this. Maybe because the fedora-review flag was set? Try it now. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=7AHyM6VzBea=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #25 from Christopher Meng cicku...@gmail.com --- (In reply to T.C. Hollingsworth from comment #24) (In reply to Christopher Meng from comment #23) I'm sorry for some reason I can't take this. Maybe because the fedora-review flag was set? Try it now. No, I don't take it because I have no time now. There is a swap in devel, I think you can consider about swapping... -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=FyP8KhOLP9a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #21 from T.C. Hollingsworth tchollingswo...@gmail.com --- (In reply to Christopher Meng from comment #20) TC I think you can close the ticket on rpmfusion. If you look more closely you'll notice that the only open review there is for an add-on package to this one. Please keep discussions about third-party repositories in their own bug trackers. They might involve issues that shouldn't be discussed here. ;-) BTW what's the progress here? Sorry, forgot to update this. -- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.9.8-1.fc19.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5687291 * Thu Aug 01 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.9.8-1 - new upstream release 0.9.8 http://lists.devloop.org.uk/pipermail/shifter-users/2013-July/000615.html - use HTTPS for URL and Source0 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=NX4DnNus4La=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Christopher Meng cicku...@gmail.com changed: What|Removed |Added CC||cicku...@gmail.com Flags||needinfo?(kvo...@redhat.com ||) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=3fSrJJEKd9a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Blocks||990805 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=JnNUSmR6Sva=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #20 from Christopher Meng cicku...@gmail.com --- TC I think you can close the ticket on rpmfusion. BTW what's the progress here? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2kYwwYrm0Fa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #19 from Antoine Martin anto...@nagafix.co.uk --- More bugfixes today (some more important ones this time around). Just (wrongly) posted this on the rpmfusion ticket: I haven't tested systemd integration, but others have: https://wiki.archlinux.org/index.php/Xpra#Server -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=O6lFsYQQVha=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #18 from T.C. Hollingsworth tchollingswo...@gmail.com --- Upstream made some more minor bugfixes. -- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.9.2-1.fc19.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5375605 * Mon May 13 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.9.2-1 - new upstream release 0.9.2 http://lists.devloop.org.uk/pipermail/shifter-users/2013-May/000525.html -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=rH7AkFoXmZa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 Bug 928609 depends on bug 953701, which changed state. Bug 953701 Summary: Review Request: python-webm - Python wrapper to WebM libraries https://bugzilla.redhat.com/show_bug.cgi?id=953701 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=1L9a7ENgWVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 Bug 928609 depends on bug 953699, which changed state. Bug 953699 Summary: Review Request: python-rencode - Web safe object pickling/unpickling https://bugzilla.redhat.com/show_bug.cgi?id=953699 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=e5YQlYq6GQa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #17 from T.C. Hollingsworth tchollingswo...@gmail.com --- No worries, it took that three weeks to ready the deps anyway. ;-) Next week is fine. In the meantime, here's a minor upstream bugfix update. -- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.9.1-1.fc19.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5362810 * Fri May 10 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.9.1-1 - new upstream release 0.9.1 http://lists.devloop.org.uk/pipermail/shifter-users/2013-May/000522.html -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=V6K6JuStAea=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #16 from Karel Volný kvo...@redhat.com --- Hi, I'm sorry but I got buried in some other stuff, if anyone is able to pick up this feel free to do so, otherwise I'll take a look again not until the next week (which doesn't sound that far in the future, but I just don't want to leave you uninformed for such long time, it's three weeks since my last comment already ...) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=PXR3T2uZYTa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #15 from T.C. Hollingsworth tchollingswo...@gmail.com --- python-webm is now in Rawhide and updates-testing for all active branches as well. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4hzZT9iu53a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #13 from T.C. Hollingsworth tchollingswo...@gmail.com --- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.9.0-1.fc19.src.rpm * Thu May 02 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.9.0-1 - new upstream release 0.9.0 http://lists.devloop.org.uk/pipermail/shifter-users/2013-April/000479.html - delete the bundled code in prep instead of inside the patches - don't bother including parti; it's going away upstream soon - merge python-wimpiggy into main xpra package; it won't be seperated upstream soon -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=k6L1XB6CH5a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #14 from T.C. Hollingsworth tchollingswo...@gmail.com --- This update fixes a small glitch in the rencode unbundling. Also, python-rencode is in F(17|18|19) updates-testing now, and python-webm is waiting on git. -- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.9.0-2.fc19.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5338048 * Tue May 07 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.9.0-2 - fix rencode __version__ importing -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4e8BRX4pBVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #11 from Antoine Martin anto...@nagafix.co.uk --- [!]: Package contains no bundled libraries without FPC exception. ^ there is rencode.pyx - http://code.google.com/p/rencode/ FYI: xpra can be built without rencode: --without-rendode But, the performance difference is very noticeable rencode vs upstream (...) We have merged the latest changes from upstream, and added some python2.4/3.x compatibility fixes which have been sent upstream (but not heard back). and python-webm - http://code.google.com/p/python-webm/ FYI, as above: can be built without webp (aka webm): --without-webp and a part of toonplayer - https://bitbucket.org/aalex/toonplayer Has been removed from the test tree. ^ python-wimpiggy is the base package... FYI: as of this week's trunk: parti has been removed completely: we don't maintain it or use it, and no-one uses it anyway (that we know of) wimpiggy source has been merged into xpra ^ note that the file NEWS is the same for all packages, could we have it symlinked? As per above: no longer an issue ^ note that the egg-infos don't look 100% correct, e.g. parti-all has License: UNKNOWN As per above: no longer an issue -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=hF5QAFSXRBa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 Antoine Martin anto...@nagafix.co.uk changed: What|Removed |Added CC||anto...@nagafix.co.uk --- Comment #10 from Antoine Martin anto...@nagafix.co.uk --- Hi there, xpra maintainer here. Is there any particular reason why you didn't contact us at all about this? It seems to me that: * we could have benefited from the feedback * our users could have had better packages * you could/will avoid issues by discussing with us (esp with current trunk) FYI: 0.10 will be substantially different when it comes to packaging. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=aOYTt8Tp2ka=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #12 from T.C. Hollingsworth tchollingswo...@gmail.com --- (In reply to comment #10) Hi there, xpra maintainer here. Is there any particular reason why you didn't contact us at all about this? It seems to me that: * we could have benefited from the feedback * our users could have had better packages * you could/will avoid issues by discussing with us (esp with current trunk) FYI: 0.10 will be substantially different when it comes to packaging. Hi, Antoine! Sorry I didn't get in touch, this was just a little side project for me and I've been busy with other things. I'm subscribed to your mailing list now so I can keep up with changes better. Plus, TBH there have been no major issues that I felt the need to complain about. ;-) (In reply to comment #11) [!]: Package contains no bundled libraries without FPC exception. ^ there is rencode.pyx - http://code.google.com/p/rencode/ FYI: xpra can be built without rencode: --without-rendode But, the performance difference is very noticeable In Fedora we like to avoid bundling libraries wherever possible, so we need to ship rencode and webm separately. But I understand both rencode and webm are small and it would be a pain for your users to require it as a dependency. (However, if you'd be interested in doing it anyway or making it an option at build time, I'd be happy to send a patch your way.) rencode vs upstream (...) We have merged the latest changes from upstream, and added some python2.4/3.x compatibility fixes which have been sent upstream (but not heard back). In cases like this, we'd still want to keep rencode seperate, but apply your changes as a patch to that package so other consumers can benefit from your bugfixes in upstream's absence. and python-webm - http://code.google.com/p/python-webm/ FYI, as above: can be built without webp (aka webm): --without-webp Again, we really want to keep this functionality, we just need to ship the package separately. and a part of toonplayer - https://bitbucket.org/aalex/toonplayer Has been removed from the test tree. ^ python-wimpiggy is the base package... FYI: as of this week's trunk: parti has been removed completely: we don't maintain it or use it, and no-one uses it anyway (that we know of) wimpiggy source has been merged into xpra Awesome! I'll go ahead and drop parti from the 0.9.0 package I'm working on. No sense in shipping it in Fedora if it's going to go away soon. ^ note that the file NEWS is the same for all packages, could we have it symlinked? As per above: no longer an issue ^ note that the egg-infos don't look 100% correct, e.g. parti-all has License: UNKNOWN As per above: no longer an issue Thanks for the heads up! -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Ml4C3y7l2Ha=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 T.C. Hollingsworth tchollingswo...@gmail.com changed: What|Removed |Added Depends On||953669, 953701 Whiteboard|NEEDSWORK | Flags|needinfo?(tchollingsworth@g | |mail.com) | --- Comment #9 from T.C. Hollingsworth tchollingswo...@gmail.com --- It appears the version Deluge uses is included with rencode upstream, and provides the same interface in pure Python as the Cython version bundled with xpra and also included with rencode upstream. Therefore, it seems prudent just to package python-rencode, whose review is in bug 953699. I've notified the deluge maintainers and suggested using the python-rencode package in bug 953700. I also unbundled python-webm, it's in bug 953701. The other minor issues identified have also been fixed. Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.8.8-4.fc19.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5273541 * Thu Apr 18 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.8.8-4 - unbundle rencode and webm - fix equality operator in Requires - drop unnecessary multiple copies of NEWS - don't remove buildroot -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AGVkN4qEi1a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 T.C. Hollingsworth tchollingswo...@gmail.com changed: What|Removed |Added Depends On|953669 |953699 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ZmbDWQDZQja=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #6 from Karel Volný kvo...@redhat.com --- (In reply to comment #5) - install fails because of dependency on gstreamer-plugins-ugly, this belongs to the rpmfusion part - gtk-update-icon-cache and update-desktop-database not called in post scripts I fixed both of these. thanks in addition, it reports more rpmlint problems not discussed above - strange permissions: snip list I believe these should be 755 I'm not sure why rpmlint doesn't complain about these on my machine, but I nonetheless ensured chmod was run on these files in the spec. probably it has something to do with the way mock sets up the environment ... so, trying with the new version, stay tuned :-) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=VpH3u4n0Uxa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 Karel Volný kvo...@redhat.com changed: What|Removed |Added Flags||needinfo?(tchollingsworth@g ||mail.com) --- Comment #8 from Karel Volný kvo...@redhat.com --- to summarize: * bundled code - the file gl_texture_bind_test.py is based on gl-hello.py from toonplayer this is irrelevant, as it doesn't get packaged; if it would be included, the only issue with this would be to get the licensing correct - rencode - http://code.google.com/p/rencode/ hm, well, deluge seems to be using this too and it is not explicitly discussed in the review request (bug #221669) ... however, the deluge's version seems completely different, so it would be a big problem to share the code, so I would go for an exception here note that this adds GPLv3+ to licenses, and according to https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#Footnote3 I believe whole xpra would need to upgrade to GPLv3+ ...? - python-webm - http://code.google.com/p/python-webm/ I believe this is a good candidate for unbundling, but I'm not opposing another exception * rm -rf %{buildroot} we've dropped this long time ago ... I cannot find it anywhere as formal requirement not to have this, but I'd remove the line so the tool doesn't complain about that :-) * versioned dependency operator according to: http://docs.fedoraproject.org/en-US/Fedora_Draft_Documentation/0.1/html/RPM_Guide/ch-advanced-packaging.html#id709435 == is not valid operator, just = * large docs I'd suggest not to have three copies of NEWS ... -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LYHZ4yllNra=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 Karel Volný kvo...@redhat.com changed: What|Removed |Added Whiteboard||NEEDSWORK --- Comment #7 from Karel Volný kvo...@redhat.com --- Package Review == Key: [x] = Pass [!] = Fail [-] = Not applicable [?] = Not evaluated [ ] = Manual review needed ^ my notes [+] reviewed, okay Issues: === - update-desktop-database is invoked when required Note: desktop file(s) in xpra See: http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache ^ seems like a false positive - filed bug #952593 = MUST items = C/C++: [+]: Package does not contain kernel modules. [+]: Package contains no static executables. ^ find rpms-unpacked/ | xargs file | grep static [+]: Development (unversioned) .so files in -devel subpackage, if present. Note: Unversioned so-files in private %_libdir subdirectory (see attachment). Verify they are not in ld path. ^ see comments #4/#5 [x]: Package does not contain any libtool archives (.la) [x]: Rpath absent or only used for internal libs. Generic: [+]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. ^ GPLv2+ [+]: %build honors applicable compiler flags or justifies otherwise. [!]: Package contains no bundled libraries without FPC exception. ^ there is rencode.pyx - http://code.google.com/p/rencode/ and python-webm - http://code.google.com/p/python-webm/ and a part of toonplayer - https://bitbucket.org/aalex/toonplayer gl-hello.py = gl_texture_bind_test.py ^ this influences the license breakdown, rencode and toonplayer are GPLv3+ and webm is BSD-like [+]: Changelog in prescribed format. [!]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. Note: rm -rf %{buildroot} present but not required [+]: Sources contain only permissible code or content. [-]: Development files must be in a -devel package [+]: Package requires other packages for directories it uses. [+]: Package uses nothing in %doc for runtime. [+]: Package is not known to require ExcludeArch. [!]: Fully versioned dependency in subpackages, if present. Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in parti , python-wimpiggy ^ python-wimpiggy is the base package, it doesn't need to depend on anything both xpra and parti depend od python-wimpiggy, I believe that {?_isa} is not mandatory in this case the problem is the == syntax, I cannot find this documented as valid, single = should be used [!]: Package complies to the Packaging Guidelines [!]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: BSD (2 clause), *No copyright* GPL (v3 or later), GPL (v3 or later), Unknown or generated. 4 files have unknown license. Detailed output of licensecheck in /home/kvolny/Projects/928609-xpra/licensecheck.txt ^ see bundled code discussion [+]: License file installed when any subpackage combination is installed. ^ it is in wimpiggy which is required by all [+]: Package consistently uses macro is (instead of hard-coded directory names). [+]: Package is named according to the Package Naming Guidelines. [+]: Package does not generate any conflict. [+]: Package obeys FHS, except libexecdir and /usr/target. [-]: If the package is a rename of another package, proper Obsoletes and Provides are present. [+]: Package must own all directories that it creates. [+]: Package does not own files or directories owned by other packages. [+]: Requires correct, justified where necessary. [+]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [+]: gtk-update-icon-cache is invoked when required Note: icons in xpra [+]: Useful -debuginfo package or justification otherwise. [+]: Large documentation must go in a -doc subpackage. Note: Documentation size is 194560 bytes in 8 files. ^ note that the file NEWS is the same for all packages, could we have it symlinked? [x]: All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines. [x]: %config files are marked noreplace or the reason is justified. [x]: Each %files section contains %defattr if rpm 4.4 [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Package contains desktop file if it is a GUI application. [x]: Package installs a %{name}.desktop using desktop-file-install if there is such a file. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Spec file lacks Packager, Vendor, PreReq tags. [x]: If (and only if) the
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 Karel Volný kvo...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||kvo...@redhat.com Assignee|nob...@fedoraproject.org|kvo...@redhat.com Flags||fedora-review? --- Comment #4 from Karel Volný kvo...@redhat.com --- a few complaints from the fedora-review tool: - install fails because of dependency on gstreamer-plugins-ugly, this belongs to the rpmfusion part - gtk-update-icon-cache and update-desktop-database not called in post scripts in addition, it reports more rpmlint problems not discussed above - strange permissions: xpra.x86_64: E: non-standard-executable-perm /usr/lib64/python2.7/site-packages/xpra/wait_for_x_server.so 0775L xpra.x86_64: E: non-standard-executable-perm /usr/lib64/python2.7/site-packages/xpra/xor/cyxor.so 0775L xpra.x86_64: E: non-standard-executable-perm /usr/lib64/python2.7/site-packages/xpra/stats/cymaths.so 0775L xpra.x86_64: E: non-standard-executable-perm /usr/lib64/python2.7/site-packages/xpra/rencode/_rencode.so 0775L python-wimpiggy.x86_64: E: non-standard-executable-perm /usr/lib64/python2.7/site-packages/wimpiggy/lowlevel/bindings.so 0775L python-wimpiggy.x86_64: E: non-standard-executable-perm /usr/lib64/python2.7/site-packages/wimpiggy/gdk/gdk_atoms.so 0775L I believe these should be 755 also the list corresponds to the list of unversioned libraries: xpra: /usr/lib64/python2.7/site-packages/xpra/rencode/_rencode.so xpra: /usr/lib64/python2.7/site-packages/xpra/stats/cymaths.so xpra: /usr/lib64/python2.7/site-packages/xpra/wait_for_x_server.so xpra: /usr/lib64/python2.7/site-packages/xpra/xor/cyxor.so python-wimpiggy: /usr/lib64/python2.7/site-packages/wimpiggy/gdk/gdk_atoms.so python-wimpiggy: /usr/lib64/python2.7/site-packages/wimpiggy/lowlevel/bindings.so I believe this fits under the exception mentioned in https://fedoraproject.org/wiki/Packaging:Guidelines#Devel_Packages As an additional complication, some software generates unversioned shared objects which are not intended to be used as system libraries. These files are usually plugins or modular functionality specific to an application, and are not located in the ld library paths or cache. This means that they are not located directly in /usr/lib or /usr/lib64, or in a directory listed as a library path in /etc/ld.so.conf (or an /etc/ld.so.conf.d/config file). Usually, these unversioned shared objects can be found in a dedicated subdirectory under /usr/lib or /usr/lib64 (e.g. /usr/lib/purple-2/ is the plugin directory used for libpurple applications). In these cases, the unversioned shared objects do not need to be placed in a -devel package. ... right? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=uk6Q11mv6Ba=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #5 from T.C. Hollingsworth tchollingswo...@gmail.com --- Thanks for the review! (In reply to comment #4) a few complaints from the fedora-review tool: - install fails because of dependency on gstreamer-plugins-ugly, this belongs to the rpmfusion part - gtk-update-icon-cache and update-desktop-database not called in post scripts I fixed both of these. in addition, it reports more rpmlint problems not discussed above - strange permissions: snip list I believe these should be 755 I'm not sure why rpmlint doesn't complain about these on my machine, but I nonetheless ensured chmod was run on these files in the spec. also the list corresponds to the list of unversioned libraries: snip list I believe this fits under the exception mentioned in https://fedoraproject.org/wiki/Packaging:Guidelines#Devel_Packages snip quote ... right? Yes. These unversioned shared objects are C extensions to the Python interpreter, so thus fit the spirit and letter of the quoted exception. -- Spec: http://patches.fedorapeople.org/xpra/xpra.spec SRPM: http://patches.fedorapeople.org/xpra/xpra-0.8.8-3.fc18.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5243685 * Thu Apr 11 2013 T.C. Hollingsworth tchollingswo...@gmail.com - 0.8.8-3 - drop unmet dependency on gstreamer-plugins-ugly - fix permissions on shared objects - add scriptlets necessary for icon/desktop file -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=JaRGG5glgVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 Dominik 'Rathann' Mierzejewski domi...@greysector.net changed: What|Removed |Added CC||domi...@greysector.net --- Comment #2 from Dominik 'Rathann' Mierzejewski domi...@greysector.net --- I think you can enable WebM/VPX. libvpx is available in Fedora. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=u3Y3Hp15zEa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 --- Comment #3 from T.C. Hollingsworth tchollingswo...@gmail.com --- Unfortunately the vpx codec requires libswscale, which is part of ffmpeg. :-( -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=CkhxGsYD0La=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928609] Review Request: xpra - screen for X
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928609 T.C. Hollingsworth tchollingswo...@gmail.com changed: What|Removed |Added CC||a...@redhat.com --- Comment #1 from T.C. Hollingsworth tchollingswo...@gmail.com --- *** Bug 651591 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=769ZpwkQ6Na=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review