[Geany-devel] Glade 3 version?

[Nick Treleaven]

On 19/12/2011 14:40, Matthew Brush wrote:
> On 12/19/2011 05:54 AM, Nick Treleaven wrote:
>> Hi,
>> I tried opening data/geany.glade with the latest Glade, 3.8.1 on
>> Windows. Pressing Save writes a lot of changes to the file, 260 Kb. It
>> seems to be mostly reordering property tag lines for
>> "use_action_appearance".
>>
>> Also, when I added 2 menu items it created duplicate image ids for
>> image1, image2.
>
> Not sure what this means.

It stopped Geany from starting with an error message about image1 being 
defined twice.

>> What version of Glade created the file? Maybe if I can use the same
>> version I won't get the reordering noise in the diff.
>>
>
> I can't remember 100% but I think it was either 3.6.x or 3.8.x. I did
> however just make this[1] change with 3.8.0 for sure recently.

Ok, there doesn't seem to be a 3.8.0 binary for Windows. The 3.6 binary 
had issues on my old machine so I'd rather stay on 3.8.1. It includes a 
relevant fix:
	- Ensure 'use-action-appearance' is serialized
	  before 'related-action' (bug 658497)

This is probably the cause of 99% of the diff.

> IMO, if you can't get the changes down to less, it doesn't really matter
> if there's a bunch of noise in the commit, it's not like anyone really
> needs to be able to read it, as long as the commit message describes

I disagree no one needs to read it. glade diffs should be reviewed by 
the author the same as all other code checkins IMO.

I'm hoping that we can standardise on 3.8.1 so we can review diffs and 
also avoid adding noise/bloat to the git repo each time someone uses a 
different version of Glade than the last commit.

Before we decided to standardise on a glade version for 2.x to prevent 
the same problem (although this was also because 2.x didn't have a 
required gtk option, different versions added a lot of diff noise anyway).

> what was changed and it's not just Windows munging the file (with \r\n,
> etc.).

I always choose the default core.autocrlf(?) option for msys-git as 
recommended by github, and the diff was produced by git.