Groovy
  1. Groovy
  2. GROOVY-4306

Groovy Console Icon for Mac OS X broken

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.7.3, 1.7.4
    • Fix Version/s: 1.8.1, 1.9-beta-1, 1.7.11
    • Component/s: Groovy Console
    • Labels:
      None
    • Environment:
      Mac OS X

      Description

      The icns file includes in groovy-1.7.3 seems to be broken. Find mine enclosed

      1. groovy.icns
        99 kB
      2. groovy.icns
        108 kB
        Tim Yates
      3. groovyicon.psd
        387 kB
        Tim Yates

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        370d 20h 50m 1 Guillaume Delcroix 21/Jul/11 06:29
        Resolved Resolved Closed Closed
        13h 37m 1 Paul King 21/Jul/11 20:06
        Mark Thomas made changes -
        Workflow jira [ 12970729 ] Default workflow, editable Closed status [ 12978502 ]
        Mark Thomas made changes -
        Project Import Mon Apr 06 02:11:23 UTC 2015 [ 1428286283443 ]
        Mark Thomas made changes -
        Workflow jira [ 12733249 ] Default workflow, editable Closed status [ 12744876 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 13:32:57 UTC 2015 [ 1428240777691 ]
        Paul King made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Tim Yates made changes -
        Attachment groovyicon.psd [ 56081 ]
        Hide
        Tim Yates added a comment -

        Cool Fingers crossed!

        Here's the PSD (and I'm not precious about this sort of thing, so I really don't mind if someone more visual than me uses it, changes it, or replaces it with something else

        Show
        Tim Yates added a comment - Cool Fingers crossed! Here's the PSD (and I'm not precious about this sort of thing, so I really don't mind if someone more visual than me uses it, changes it, or replaces it with something else
        Hide
        Guillaume Delcroix added a comment -

        Yeah, the PSD, we could put it in the graphics folder of SVN.

        Regarding the build, we have this which should take care of not chewing the icns file hopefully:

        <fixcrlf srcdir="${installDirectory}/bin" eol="lf" excludes="*.bat, *.icns"/>
        
        Show
        Guillaume Delcroix added a comment - Yeah, the PSD, we could put it in the graphics folder of SVN. Regarding the build, we have this which should take care of not chewing the icns file hopefully: <fixcrlf srcdir= "${installDirectory}/bin" eol= "lf" excludes= "*.bat, *.icns" />
        Hide
        Tim Yates added a comment -

        Should I attach the psd?

        I think it's something between building the install packages, and pushing the release that is chewing up the icns file, but I have no idea what

        Fingers crossed though

        Show
        Tim Yates added a comment - Should I attach the psd? I think it's something between building the install packages, and pushing the release that is chewing up the icns file, but I have no idea what Fingers crossed though
        Guillaume Delcroix made changes -
        Fix Version/s 1.8.1 [ 17223 ]
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.7.11 [ 17244 ]
        Fix Version/s 1.9-beta-1 [ 17153 ]
        Hide
        Guillaume Delcroix added a comment -

        Hopefully with the new icon the problem's gone.
        Thanks Tim for the graphics!

        Show
        Guillaume Delcroix added a comment - Hopefully with the new icon the problem's gone. Thanks Tim for the graphics!
        Guillaume Delcroix made changes -
        Assignee Andres Almiray [ aalmiray ] Guillaume Laforge [ guillaume ]
        Tim Yates made changes -
        Attachment groovy.icns [ 56080 ]
        Hide
        Tim Yates added a comment - - edited

        Still broken in 1.7.8 and 1.8.0

        I created a new groovy.icns file (attached) as well, as the existing icon shows some harsh scaling artifacts when above 32x32 px in size...

        Show
        Tim Yates added a comment - - edited Still broken in 1.7.8 and 1.8.0 I created a new groovy.icns file (attached) as well, as the existing icon shows some harsh scaling artifacts when above 32x32 px in size...
        Jochen Theodorou made changes -
        Fix Version/s 1.7.6 [ 16749 ]
        Fix Version/s 1.8-beta-3 [ 16750 ]
        Guillaume Delcroix made changes -
        Fix Version/s 1.7.5 [ 16639 ]
        Fix Version/s 1.8-beta-3 [ 16750 ]
        Fix Version/s 1.8-beta-2 [ 16629 ]
        Fix Version/s 1.7.6 [ 16749 ]
        Hide
        Guillaume Delcroix added a comment -

        Andres, did you see what garbles the icon?

        Show
        Guillaume Delcroix added a comment - Andres, did you see what garbles the icon?
        Guillaume Delcroix made changes -
        Fix Version/s 1.8-beta-2 [ 16629 ]
        Affects Version/s 1.7.4 [ 16563 ]
        Fix Version/s 1.7.5 [ 16639 ]
        Hide
        Ingo Richter added a comment -

        The groovy.icns is broken with 1.7.4 as well.

        Show
        Ingo Richter added a comment - The groovy.icns is broken with 1.7.4 as well.
        Guillaume Delcroix made changes -
        Field Original Value New Value
        Assignee Andres Almiray [ aalmiray ]
        Hide
        Stefan Scheidt added a comment -

        Replaced my local groovy.icns with this one: http://fisheye.codehaus.org/browse/groovy/trunk/groovy/groovy-core/src/bin/groovy.icns.
        That solves the "problem".

        Show
        Stefan Scheidt added a comment - Replaced my local groovy.icns with this one: http://fisheye.codehaus.org/browse/groovy/trunk/groovy/groovy-core/src/bin/groovy.icns . That solves the "problem".
        Hide
        Stefan Scheidt added a comment -

        See also GROOVY-4183

        Show
        Stefan Scheidt added a comment - See also GROOVY-4183
        Stefan Scheidt created issue -

          People

          • Assignee:
            Guillaume Delcroix
            Reporter:
            Stefan Scheidt
          • Votes:
            2 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development