Spec URL: https://ignatenkobrain.fedorapeople.org/for-review/screencloud.spec SRPM URL: https://ignatenkobrain.fedorapeople.org/for-review/screencloud-1.3.1-1.fc26.src.rpm Description: Easy to use screenshot sharing application. Take a screenshot using one of the 3 hotkeys or simply click the ScreenCloud tray icon. You can choose where you want to save your screenshot. If you choose to upload your screenshot to the screencloud website, a link will automatically be copied to your clipboard. You can share this link with your friends or colleagues via email or in an IM conversation. All they have to do is click the link and look at your screenshot. ScreenCloud also offers FTP and SFTP uploading if you want to use your own server. Fedora Account System Username: ignatenkobrain
Created attachment 1236717 [details] Patch for unbundling qtsingleapplication I'd justify bundling libqxt quoting upstream: " LIBQXT IS NO LONGER MAINTAINED Qxt will likely not work with newer Qt versions due to usage of internal api. We recommend that you pick out the parts you want instead of using the entire libqxt. " Patch for unbundling qtsingleapplication attached.
Hi Igor, any update here?
(In reply to Sandro Mani from comment #2) > Hi Igor, any update here? Hey, give me couple of days. Busy =(
No worries, just asking ;)
Hello, According to the fedora-review you dont have to list "gcc" and "make" in BuildRequires, i compiled the code after removal and it works so i think you can remove it. Upstream already accepted change of desktop file and removed execution permission from LICENSE and README.md so you can remove that parts too.
And if you would have some time you could also look at the unbundling.
(In reply to Tomáš Korbař from comment #5) > Hello, > According to the fedora-review you dont have to list "gcc" and "make" in > BuildRequires, i compiled the code after removal and it works so i think you > can remove it. > Upstream already accepted change of desktop file and removed execution > permission from LICENSE and README.md so you can remove that parts too. Bad luck, fedora-review. Guidelines say to explicitly mention everything. (In reply to Tomáš Korbař from comment #6) > And if you would have some time you could also look at the unbundling. I tried to, but I failed (at least on libxqt), and qtsingleapplication failed too because they patch code.
This is an automatic check from review-stats script. This review request ticket hasn't been updated for some time, but it seems that the review is still being working out by you. If this is right, please respond to this comment clearing the NEEDINFO flag and try to reach out the submitter to proceed with the review. If you're not interested in reviewing this ticket anymore, please clear the fedora-review flag and reset the assignee, so that a new reviewer can take this ticket. Without any reply, this request will shortly be resetted.
This is an automatic action taken by review-stats script. The ticket reviewer failed to clear the NEEDINFO flag in a month. As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews we reset the status and the assignee of this ticket.
Seems like the submitter account is disabled (?), please make sure you have added the correct RHBZ (Red Hat Bugzilla) email. I will be closing this package review if there is no response in a week.