After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 735275 - gnome-icon-themes dependency no longer valid with GNOME 3.14
gnome-icon-themes dependency no longer valid with GNOME 3.14
Status: RESOLVED FIXED
Product: gupnp-tools
Classification: Other
Component: General
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GUPnP Maintainers
GUPnP Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-08-23 10:58 UTC by Dominique Leuenberger
Modified: 2019-02-22 05:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Do not depend on gnome-icon-theme (1.28 KB, patch)
2014-08-23 10:58 UTC, Dominique Leuenberger
committed Details | Review
common: Change icon loading behaviour (4.91 KB, patch)
2014-08-25 20:30 UTC, Jens Georg
committed Details | Review

Description Dominique Leuenberger 2014-08-23 10:58:04 UTC
gnome-icon-themes has been merged and renamed to adwaita-icon-theme

In fact, the build dep on g-i-t by gupnp-tools looks bogus anyway - nothing is required of it (it was probably introduced to ensure users that build it from source have it installed)
Comment 1 Dominique Leuenberger 2014-08-23 10:58:38 UTC
Created attachment 284283 [details] [review]
Do not depend on gnome-icon-theme
Comment 2 Jens Georg 2014-08-24 07:41:53 UTC
av-cp and universal-cp used to abort when they could not find their icons. I don't know if that's still the case after the UI fiddling.
Comment 3 Dominique Leuenberger 2014-08-24 10:32:01 UTC
(In reply to comment #2)
> av-cp and universal-cp used to abort when they could not find their icons. I
> don't know if that's still the case after the UI fiddling.

Fair point - but gnome-ion-themes still no longer exists :)

gupnp-av-cp seems to make references to icons like gtk-execute, which is not part of adwaita-icon-theme...

so there is some more work required - but soon, as 3.14.0 does not come with gnome-icon-themes-* anymore.
Comment 4 Jens Georg 2014-08-25 08:41:02 UTC
Agree, I was just giving an explanation for that check :)
Comment 5 Jens Georg 2014-08-25 19:23:21 UTC
Hm, gtk-execute is the "old" stock id. is this really deprecated?
Comment 6 Jens Georg 2014-08-25 19:25:36 UTC
I think we need to have a deeper look at the icon code.
Comment 7 Dominique Leuenberger 2014-08-25 19:55:23 UTC
(In reply to comment #6)
> I think we need to have a deeper look at the icon code.

Checked on what there was in gnome-icon-theme (up to 3.12.0):

src/Makefile.am contained:
        if [ $(ICONMAP) != "false" ]; then \
                cd $(DESTDIR)$(themedir); \
                for size in $(render_sizes); do \
                        cd $$size || continue; \
                        echo -e "Going to set up icon mapping for $$size"; \
                        for dir in `find . -type d`; do \
                                context="`echo $$dir | cut -c 3-`"; \
                                if [ $$context ]; then \
                                        $(ICONMAP) -c $$context; \
                                fi; \
                        done; \
                        cd ..; \
                done; \
        fi



adwaita-icon-theme does not contain any such mapping code - which results in the missing icon compatibility names.
Comment 8 Jens Georg 2014-08-25 20:30:40 UTC
Created attachment 284451 [details] [review]
common: Change icon loading behaviour

Signed-off-by: Jens Georg <mail@jensge.org>
Comment 9 Jens Georg 2014-08-27 20:45:24 UTC
Attachment 284451 [details] pushed as 510c1ca - common: Change icon loading behaviour