← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] 30 icons updated + new ico files generated

 

well guys I do not really know what the problem is. Once I kind of got
the impression that a fresh branch would rebuild all PNGs if the PNG
MAINTAINER flag was on, even if not .svg were changed.
I will certainly be more carefully in the future when building new
.svg. Anyways, in a little while, I don't think anybody will spend so
much time with these svg files anyway.

Cheers
Fabrizio


On Wed, Sep 14, 2011 at 8:54 PM, jean-pierre charras
<jp.charras@xxxxxxxxxx> wrote:
> Le 14/09/2011 20:24, Dick Hollenbeck a écrit :
>>
>> On 09/14/2011 06:19 AM, fabrizio wrote:
>>>
>>> Hello,
>>> In this patch the following updates were made:
>>> - 30 icons/buttons were redrawn in accordance to various suggestions
>>> - updated all .ico images so that they now contain high resolution
>>> images (good for OS X)
>>> - a script to generate .ico file was added
>>> - some redundant buttons (like zoomoins3d.svg ) were deleted
>>>
>>> Regards
>>> Fabrizio
>>> PS sorry for the big patch but I do not understand how to not include
>>> the rebuilt .cpp icons. the touch suggestion did not seem to work.
>>
>> I wonder what the delta is on the *.cpp files, even if they were
>> re-written?
>>
>> If the PNG file is generated exactly the same on two runs, I would think
>> that
>> "bzr diff" should not see this as a change in the *.cpp files.
>>
>> So we are perhaps still bringing some extraneous gunk into the PNG file,
>> like a
>> date stamp or something?
>>
>> Dick
>>
>
>
> Previous .cpp files were rebuilt by me (because fabrizio sent me only .svg
> files).
> To avoid a lot of .cpp changes, I rebuilt .cpp files before commit the last
> patch from fabrizio.
>
> Seems a lot of changes were removed.
> I had a look to the patch file:
> Perhaps there are some changes due Inkscape or pngcrush version differences.
> In fact I am pretty sure differences are mainly due to End Of Line
> characters (LF versus CRLF)
>
> Usually I do not have this problem when editing Kicad sources,
> because I am using the same editor under Window and Linux:
> Scite, a multi-platform FOSS very good editor) that handle very well EOL
> format.
> It uses LF when opening a text file using LF and automatically uses CRLF if
> the text uses this format,
> and it can easily convert the existing EOL format to LF, CRLF or CR format.
>
>
> --
> Jean-Pierre CHARRAS
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>


References