Modify

Opened 3 years ago

Closed 3 years ago

#4792 closed defect (invalid)

JOSM build: fragile filtering of icons

Reported by: Gubaer Owned by: team
Priority: major Component: Core
Version: latest Keywords: icon handling, josm build
Cc:

Description

When JOSM is built by the non-public nighly build script, "unused" icons are removed from the distribution. Probably the build script uses geticons.pl.

This is a fragile concept because plugin authors rely on the icons in the JOSM repository too and because every missing icon triggers a fatal exception.

Eiter

  • include all icon below images

or

  • scan dependencies in the plugins managed on the OSM SVN too

Attachments (0)

Change History (4)

comment:1 Changed 3 years ago by stoecker

  • Resolution set to invalid
  • Status changed from new to closed

Build does not filter icons in main JOSM. It only filters icons used for the internal style. Everything else is still hand-made. The style-icons do not cause fatal errors when missing.

comment:2 Changed 3 years ago by stoecker

P.S. The style subdirectory is out of control for josm, so this directory is not really usable for program code and must be filtered due to much unused stuff.

comment:3 Changed 3 years ago by Gubaer

  • Resolution invalid deleted
  • Status changed from closed to reopened

Not solved yet. A plugins still runs a in fatal exception if it loads an icon from the images/styles which is filtered out by the nighly build scripts. Again: the script which checks dependencies and silently drops images from the release, can and should check dependencies in the known plugins too.

comment:4 Changed 3 years ago by anonymous

  • Resolution set to invalid
  • Status changed from reopened to closed

Everything in styles changes from time to time without any influence from josm development. Sometimes icons are rearranged there completely, moved or renamed. As already said this should not be used in JOSM code, only for icons which can be missing (like style).

It is a bug when style-icons are used in JOSM code.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.