[Bug 1582469] Review Request: plasma-browser-integration - components necessary to integrate browsers into the Plasma Desktop

[bugzilla]

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

Jan Grulich  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2019-01-02 10:15:24



-- 
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://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org

[Bug 1582469] Review Request: plasma-browser-integration - components necessary to integrate browsers into the Plasma Desktop

[bugzilla]

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



--- Comment #4 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/plasma-browser-integration

-- 
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://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/JUWZ6LA4RKIRYL3ZTXFCUWME7TACADPI/

[Bug 1582469] Review Request: plasma-browser-integration - components necessary to integrate browsers into the Plasma Desktop

[bugzilla]

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



--- Comment #3 from Kevin Kofler  ---
Shouldn't the %description mention the browsers that are actually supported by
this thing? Because there are definitely browsers out there that it won't work
with.

-- 
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://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/NJSILQTUBMV5U2TXAVTRTTSSD54P6CDB/

[Bug 1582469] Review Request: plasma-browser-integration - components necessary to integrate browsers into the Plasma Desktop

[bugzilla]

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

Rex Dieter  changed:

   What|Removed |Added

 CC||rdie...@gmail.com
 Blocks||656997 (kde-reviews)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=656997
[Bug 656997] kde-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/7F2VXZR2B7U5CAGADDZNTK2H7ANTE763/

[Bug 1582469] Review Request: plasma-browser-integration - components necessary to integrate browsers into the Plasma Desktop

[bugzilla]

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

Robert-André Mauchin  changed:

   What|Removed |Added

 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|jgrul...@redhat.com



-- 
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://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/M6QAQNCLDSN7R4H424ZZXB5YYIDPVMSO/

[Bug 1582469] Review Request: plasma-browser-integration - components necessary to integrate browsers into the Plasma Desktop

[bugzilla]

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



--- Comment #2 from Martin Kyral  ---
Moved the patch to downstream patches:

http://copr-dist-git.fedorainfracloud.org/cgit/mkyral/plasma-unstable/plasma-browser-integration.git/commit/?id=c52348c3bcae5b80a2cc1c44bb7c89a26dae6621

-- 
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://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/WCVORLKEP53TLC5QP4RDLCB756BEJPEX/

[Bug 1582469] Review Request: plasma-browser-integration - components necessary to integrate browsers into the Plasma Desktop

[bugzilla]

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

Jan Grulich  changed:

   What|Removed |Added

 CC||jgrul...@redhat.com
  Flags||fedora-review+



--- Comment #1 from Jan Grulich  ---
1) Naming: ok
2) License: Ok, some files are GPLv2+ and some GPLv3
3) URL: ok
4) Scriptlets: ok, not used and not needed
5) Files section: ok, license file installed

Minor thing, the patch is not upstreamable, I tried to propose it to upstream,
but this way it would break installation for Debian/Ubuntu.

-- 
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://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/NZCNW2PNIG5HTGJ62AQ2B7RNP3WF3ICY/