groovy
  1. groovy
  2. GROOVY-4306

Groovy Console Icon for Mac OS X broken

    Details

    • Type: Bug Bug
    • Status: Closed 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
    • Number of attachments :
      3

      Description

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

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

        Activity

        Hide
        Stefan Scheidt added a comment -

        See also GROOVY-4183

        Show
        Stefan Scheidt added a comment - See also GROOVY-4183
        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
        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.
        Hide
        Guillaume Laforge added a comment -

        Andres, did you see what garbles the icon?

        Show
        Guillaume Laforge added a comment - Andres, did you see what garbles the icon?
        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...
        Hide
        Guillaume Laforge added a comment -

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

        Show
        Guillaume Laforge added a comment - Hopefully with the new icon the problem's gone. Thanks Tim for the graphics!
        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
        Hide
        Guillaume Laforge 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 Laforge 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 -

        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

          People

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

            Dates

            • Created:
              Updated:
              Resolved: