[gimp/wip/Jehan/installer-to-windows-store: 5/5] data: fix gimp-release file on Windows.




commit d0609f3eab28d8ff30272e22ee3a88222e2a87a6
Author: Jehan <jehan girinstud io>
Date:   Tue Jun 7 15:56:25 2022 +0200

    data: fix gimp-release file on Windows.
    
    The '\n' newline is transformed into a space (0x20) when building
    natively on Windows. And unfortunately g_key_file_load_from_file() does
    not like this.
    
    The autotools rule work properly according to my tests.
    
    Note that at first, I thought it was a LF vs. CRLF issue, and tried to
    special-case Windows, but I realized it's just that 'echo' command
    seemingly transform newlines unexpectedly in its Windows form. I didn't
    want to waste too much time experimenting alternative calls, especially
    as I have to test through CI (so each test is very costly, time-wise).
    Instead I go the alternative way of using a pre-generated conf file.

 data/gimp-release.in |  2 ++
 data/meson.build     | 22 +++++++---------------
 2 files changed, 9 insertions(+), 15 deletions(-)
---
diff --git a/data/gimp-release.in b/data/gimp-release.in
new file mode 100644
index 0000000000..904317aef3
--- /dev/null
+++ b/data/gimp-release.in
@@ -0,0 +1,2 @@
+[package]
+revision=@REVISION@
diff --git a/data/meson.build b/data/meson.build
index a9bfe63882..159ebdcd96 100644
--- a/data/meson.build
+++ b/data/meson.build
@@ -15,22 +15,14 @@ meson.add_install_script('sh', '-c',
   'mkdir -p $MESON_INSTALL_DESTDIR_PREFIX/@0@'.format(gimpdatadir / 'fonts')
 )
 
-# It's crappy, but is a combination of meson not transforming \n into
-# Windows CRLF for us (which might not be considered a bug) and GLib's
-# g_key_file_load_from_file() API apparently not able to work with Unix
-# newlines LF-only while on Windows.
-if platform_windows
-  nl='\r\n'
-else
-  nl='\n'
-endif
+release_conf = configuration_data()
+release_conf.set('REVISION', get_option('revision'))
 
-custom_target('gimp-release',
-  output: [ 'gimp-release' ],
-  command: [
-    'echo', ['[package]' + nl + 'revision=@0@'.format(get_option('revision'))]
-  ],
-  capture: true,
+gimp_release = 'gimp-release'
+configure_file(
+  input : gimp_release + '.in',
+  output: gimp_release,
+  configuration: release_conf,
   install: true,
   install_dir: gimpdatadir,
 )


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]