[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Didik Supriadi  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
 Blocks||201449 (FE-DEADREVIEW)
Last Closed||2021-08-21 15:51:54





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=201449
[Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response
should be blocking this bug.
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Didik Supriadi  changed:

   What|Removed |Added

 CC||didiksupriad...@gmail.com



--- Comment #10 from Didik Supriadi  ---
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.


-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

needinfo canceled: [Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

Product: Fedora
Version: rawhide
Component: Package Review

Package Review  has canceled Package
Review 's request for Tomáš Korbař
's needinfo:
Bug 1396847: Review Request: screencloud - Easy to use screenshot sharing
application
https://bugzilla.redhat.com/show_bug.cgi?id=1396847



--- Comment #9 from Package Review  ---
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.
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Petr Pisar  changed:

   What|Removed |Added

 CC||ppi...@redhat.com
  Flags||fedora-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847



--- Comment #7 from Igor Gnatenko  ---
(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.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847



--- Comment #6 from Tomáš Korbař  ---
And if you would have some time you could also look at the unbundling.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847



--- Comment #5 from Tomáš Korbař  ---
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.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Tomáš Korbař  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||tomas.k...@seznam.cz
   Assignee|nob...@fedoraproject.org|tomas.k...@seznam.cz



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Sandro Mani  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 CC|manisan...@gmail.com|
   Assignee|manisan...@gmail.com|nob...@fedoraproject.org
  Flags|fedora-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847



--- Comment #4 from Sandro Mani  ---
No worries, just asking ;)

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847



--- Comment #3 from Igor Gnatenko  ---
(In reply to Sandro Mani from comment #2)
> Hi Igor, any update here?
Hey, give me couple of days. Busy =(

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847



--- Comment #2 from Sandro Mani  ---
Hi Igor, any update here?

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847



--- Comment #1 from Sandro Mani  ---
Created attachment 1236717
  --> https://bugzilla.redhat.com/attachment.cgi?id=1236717=edit
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.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Sandro Mani  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||manisan...@gmail.com
   Assignee|nob...@fedoraproject.org|manisan...@gmail.com
  Flags||fedora-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Randy Barlow  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|ra...@electronsweatshop.com |nob...@fedoraproject.org
  Flags|fedora-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Randy Barlow  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ra...@electronsweatshop.com
   Assignee|nob...@fedoraproject.org|ra...@electronsweatshop.com
  Flags||fedora-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847
Bug 1396847 depends on bug 1396846, which changed state.

Bug 1396846 Summary: [PATCH] Update to 3.1; Switch to Qt5; Switch to Python3
https://bugzilla.redhat.com/show_bug.cgi?id=1396846

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847
Bug 1396847 depends on bug 1394149, which changed state.

Bug 1394149 Summary: pythonqt: Switch to Python 3
https://bugzilla.redhat.com/show_bug.cgi?id=1394149

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||screencloud



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1396847] Review Request: screencloud - Easy to use screenshot sharing application

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1396847

Igor Gnatenko  changed:

   What|Removed |Added

 Depends On||1396846, 1394149




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1394149
[Bug 1394149] pythonqt: Switch to Python 3
https://bugzilla.redhat.com/show_bug.cgi?id=1396846
[Bug 1396846] [PATCH] Update to 3.1; Switch to Qt5; Switch to Python3
-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org