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



--- Comment #6 from Neal Gompa <ngomp...@gmail.com> ---
(In reply to Brian Exelbierd from comment #5)
> Another update:
> 
> 1. I fixed the icon file name
> 2. Also, the upstream finally put the .desktop and icon somewhere we can
> reference so I fixed that.
> 3. I am providing name corrected urls based on the new name:
> 
> Spec URL: https://fedorapeople.org/~bex/DayJournal/dayjournal.spec
> 
> SRPM URL:
> https://fedorapeople.org/~bex/DayJournal/dayjournal-23.0.6-1.fc24.src.rpm
> 
> COPR is also updated.
> 
> Thank you.

Please remove the configure.patch you have that removes appindicator support.
There are desktops in Fedora (notably GNOME and Plasma) that do actually
support this, and we do have the library in our repositories.

-- 
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

Reply via email to