<p></p>
<p><b>@techee</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/3194#discussion_r867490774">meson_options.txt</a>:</p>
<pre style='color:#555'>> @@ -6,3 +6,4 @@ option('api-docs', type : 'feature', description : 'enable to generate API docum
 option('gtkdoc', type : 'boolean', description : 'enable to generate gtk-doc compatible headers for the API')
 option('python-command', type: 'string', description: 'the default Python command')
 option('socket', type: 'boolean', description: 'enable if you want to detect a running instance')
+option('mac-integration', type: 'boolean', value: 'false', description: 'enable for improved macOS integration using the gtk-mac-integration library')
</pre>
<blockquote>
<p dir="auto">Probably in-line with autotools but I wonder why this is off by default? Couldn't this default to on and maybe dynamically detect presence of the dependency (or simply require it unless mac-integration is explicitly disabled)?</p>
</blockquote>
<p dir="auto">At least for testing, I'd like to be able to manually override the auto-detected presence of the library. With autotools I just gave up ;-). I tried the following with meson:</p>
<div class="highlight highlight-source-meson"><pre>mac_integration_dep <span class="pl-k">=</span> [<span class="pl-s"><span class="pl-pds">'</span>gtk-mac-integration<span class="pl-pds">'</span></span>, <span class="pl-s"><span class="pl-pds">'</span>3.0.1<span class="pl-pds">'</span></span>]
mac_integration <span class="pl-k">=</span> <span class="pl-c1">dependency</span>(mac_integration_dep[<span class="pl-c1">0</span>], <span class="pl-smi">version</span>: <span class="pl-s"><span class="pl-pds">'</span>>= <span class="pl-pds">'</span></span> <span class="pl-k">+</span> mac_integration_dep[<span class="pl-c1">1</span>],
        <span class="pl-smi">required</span>: <span class="pl-c1">get_option</span>(<span class="pl-s"><span class="pl-pds">'</span>mac-integration<span class="pl-pds">'</span></span>).enabled())
deps_for_pc <span class="pl-k">+=</span> <span class="pl-s"><span class="pl-pds">'</span> <span class="pl-pds">'</span></span> <span class="pl-k">+</span> mac_integration_dep[<span class="pl-c1">0</span>] <span class="pl-k">+</span> <span class="pl-s"><span class="pl-pds">'</span> >= <span class="pl-pds">'</span></span> <span class="pl-k">+</span> mac_integration_dep[<span class="pl-c1">1</span>]</pre></div>
<p dir="auto">assuming (based on the documentation) that <code class="notranslate">get_option('mac-integration').enabled()</code> would return <code class="notranslate">true</code> if explicitly enabled on the command-line and <code class="notranslate">false</code> if disabled or not specified. But I get</p>
<pre class="notranslate"><code class="notranslate">meson.build:27:0: ERROR: Unknown method "enabled" in object <[BooleanHolder] holds [bool]: True> of type BooleanHolder.
</code></pre>
<p dir="auto">What am I doing wrong?</p>
<p dir="auto">In any case, I don't think auto-enabling the library is that important. The library is probably not used by many applications and not normally installed and in fact, if Geany started using GtkApplication and GMenuModel, we could eliminate this dependency as its functionality is present in GTK now.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/3194#discussion_r867490774">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ7TIJA7VP6ZNOPFRD3VI65UDANCNFSM5VGPSQQA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAIOWJYYV6ABZZK7YA3AW73VI65UDA5CNFSM5VGPSQQKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOHGFXGGY.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><geany/geany/pull/3194/review/965440283</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/3194#discussion_r867490774",
"url": "https://github.com/geany/geany/pull/3194#discussion_r867490774",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>