<p></p>
<blockquote>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/eht16/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/eht16">@eht16</a> In fact, <code>gawk</code> is not a new depedency, it's already needed by <code>geany-i18n-m4</code>. but I still replaced the offending statement because meson discourages from listing source files via wildcards.</p>
</blockquote>
<p>Almost. <code>geany-i18n-m4</code> requires <code>AC_PROG_AWK</code> which supports also other <code>awk</code> variants than <code>gawk</code>.<br>
I noticed this because the system where I tested the build actually had no <code>gawk</code> installed but I was still able to successfully compile Geany with autotools (probably with <code>mawk</code> as this is installed on that system).<br>
Maybe Meson supports something similar to use either <code>gawk</code> or <code>mawk</code> depending on what is available, so we could still have the dynamically generated list.</p>
<p>Found a few more issues:</p>
<ul>
<li>
<p><code>$PREFIX/share/geany/filedefs/filetypes.python.in</code> is installed as is. But it needs to be preprocessed to replace <code>@PYTHON_COMMAND@</code>.</p>
</li>
<li>
<p>G_LOG_DOMAIN is not set which results in log messages without the source set, e.g.: <code>17:53:15: (null) INFO           : Geany 1.38 (git >= 73ab53541), en_GB.UTF-8</code>. Log messages should look like: <code>17:54:09: Geany INFO            : Geany 1.38 (git >= 8a30ce2), en_GB.UTF-8</code>. For the different parts of the code, there are different log domains defined:<br>
CTags<br>
Classbuilder<br>
Demoplugin<br>
Demoproxy<br>
Export<br>
FileBrowser<br>
Geany<br>
HTMLChars<br>
MIO<br>
SaveActions<br>
SplitWindow<br>
Tagmanager</p>
</li>
<li>
<p>in the output I found <code>Running custom install script '/usr/bin/meson --internal gettext install --subdir=po --localedir=share/locale --pkgname=geany --langs=ar@@ast@@be@@bg@@ca@@cs@@da@@de@@el@@en_GB@@es@@et@@eu@@fa@@fi@@fr@@gl@@he@@hi@@hu@@id@@ie@@it@@ja@@kk@@ko@@ku@@lb@@lt@@lv@@mn@@nl@@nn@@pl@@pt@@pt_BR@@ro@@ru@@sk@@sl@@sr@@sv@@tr@@uk@@vi@@zh_CN@@zh_TW'</code> which looks strange but possibly is fine anyway. At least it seems the language files were probably installed.</p>
</li>
<li>
<p>I still have the messages like</p>
<pre><code>msgfmt: ../po/LINGUAS does not exist
../geany.desktop.in:5:0: warning: invalid non-blank line
../geany.desktop.in:6:0: warning: invalid non-blank line
../geany.desktop.in:7:0: warning: invalid non-blank line
../geany.desktop.in:14:0: warning: invalid non-blank line
</code></pre>
<p>I wonder if the missing <code>../po/LINGUAS</code> is a problem and what's the consequence.<br>
The lines about "warning: invalid non-blank line" seems to be related that Meson is using <code>gettext</code> to process the .desktop file where previously <code>intltool</code> was used. And the <code>_Keyword...</code> syntax is specific for <code>intltool</code>.<br>
It seems this might be a seperate task like migrating from <code>intltool</code> to <code>gettext</code> (<a href="https://stackoverflow.com/questions/62504873/i18n-merge-file-cant-translate-desktop-file" rel="nofollow">https://stackoverflow.com/questions/62504873/i18n-merge-file-cant-translate-desktop-file</a>).<br>
Anyway, we need to handle this in some way as the resulting .desktop file is missing items like "Name", "Comment" and "Keywords".</p>
</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2761#issuecomment-850972050">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJYTTQVAHC5GADHZGN3TQIC3BANCNFSM4ZDZAFLA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAIOWJZBWWDPQ526T2T5RH3TQIC3BA5CNFSM4ZDZAFLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGK4M3EQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2761#issuecomment-850972050",
"url": "https://github.com/geany/geany/pull/2761#issuecomment-850972050",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>