<br>
> On 25.01.2019, at 23:32, Eliot Miranda <notifications@github.com> wrote:<br>
> <br>
> Hi Tobias,<br>
> <br>
> On Fri, Jan 25, 2019 at 2:27 PM Tobias Pape <notifications@github.com><br>
> wrote:<br>
> <br>
> > That's besides the point here.<br>
> ><br>
> > The plugin would not have been built if the library was not present.<br>
> > That's what the configure step is for.<br>
> > The VM is origrinally from a travis and I made sure the lib is there:<br>
> > https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/scripts/ci/travis_install.sh#L15<br>
> > (thats the dev lib that depends on the real one:<br>
> > https://packages.ubuntu.com/xenial/libpulse-dev)<br>
> ><br>
> <br>
> So how come it isn't linked against libpulse-simple.so ?<br>
<br>
I have not the slightest idea and, atm, no linux at hand.<br>
We (ie, maybe, @ekrebs5 or @fniephaus) have to look wether this stems from here or https://github.com/squeak-smalltalk/squeak-app .<br>
On macOS we have rpath issues that can come out like this and might be sensitive to be moved around…<br>
<br>
Here we see that the lib is added in configure:<br>
https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/483526374#L1071<br>
<br>
And here we see that the squeak _binary_ is actually linked agains libpulse-simple:<br>
<br>
https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/483526374#L3455<br>
<br>
That is ok, since the dynloaded vm-sound-pulse can lookup in its loading process, which (being squeak being linked against libpulse-simple) should have the syms.<br>
<br>
Accordingly we see the plugin not begin linked against the lib:<br>
https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/483526374#L4111<br>
<br>
The rest is libtool chitter-chat that i cannot understand so quick…<br>
<br>
> <br>
> > All of the respecitve sound plugins are optional, and absence is expected.<br>
> ><br>
> <br>
> Not for a build slave, surely. Shouldn't;t the build slave build all<br>
> possible variants? If we're providing prebuilt VMs (for good reason) then<br>
> we have to build what we support. Things like vm-display-fbdev may be<br>
> something we can not support and leave to individuals to build if they want<br>
> to. But vm-sound-pulse is not in that category.<br>
<br>
Did not say that. all the vm-*-* plugins seem to care for optionality. and IIRC it is nigh impossible to build for some combinations to be built with the same package set (NAS + OSS or some OSS/ALSA combos, but my memory is foggy there).<br>
<br>
In fact, I tried to make sure as many as possible plugins to be buildable on Travis: <br>
https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/698661a94af388b06b98b11d9d2a4216b36acdb5#diff-852da479861e8f719725fb8018cc1699R5<br>
https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/989f7e06d041c145863dd2d648e196a3169947d4#diff-eaec0bdb68c02946e8335228ebbbbad9<br>
<br>
https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/75f59c456f79fb4f7ca64bc1d2504d3f9626e305<br>
https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/ca75c6bfd0903f25917b8b88aca63e574dd39a1f<br>
https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/a82b9cfdcf2f8d5396fa2f6950a06f35c374098d<br>
https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/3a8a3c4bb7033c0e19c714ca4d90b0cc8f9f2565<br>
<br>
Point is: yes, I really tried to make sure the build slave builds everything necessary.<br>
<br>
<br>
-t<br>
<br>
<br>
> <br>
> _,,,^..^,,,_<br>
> best, Eliot<br>
> —<br>
> You are receiving this because you were mentioned.<br>
> Reply to this email directly, view it on GitHub, or mute the thread.<br>
> <br>
<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you commented.<br />Reply to this email directly, <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/360#issuecomment-457768165">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AhLyW3k2M4o7CI3NLORKSlN_QTCLBc5Aks5vG5DLgaJpZM4aS083">mute the thread</a>.<img src="https://github.com/notifications/beacon/AhLyW_Py7yJ0UCbfeE8bpzsQrHWBUrvZks5vG5DLgaJpZM4aS083.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@krono in #360: \n\u003e On 25.01.2019, at 23:32, Eliot Miranda \u003cnotifications@github.com\u003e wrote:\n\u003e \n\u003e Hi Tobias,\n\u003e \n\u003e On Fri, Jan 25, 2019 at 2:27 PM Tobias Pape \u003cnotifications@github.com\u003e\n\u003e wrote:\n\u003e \n\u003e \u003e That's besides the point here.\n\u003e \u003e\n\u003e \u003e The plugin would not have been built if the library was not present.\n\u003e \u003e That's what the configure step is for.\n\u003e \u003e The VM is origrinally from a travis and I made sure the lib is there:\n\u003e \u003e https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/scripts/ci/travis_install.sh#L15\n\u003e \u003e (thats the dev lib that depends on the real one:\n\u003e \u003e https://packages.ubuntu.com/xenial/libpulse-dev)\n\u003e \u003e\n\u003e \n\u003e So how come it isn't linked against libpulse-simple.so ?\n\nI have not the slightest idea and, atm, no linux at hand.\nWe (ie, maybe, @ekrebs5 or @fniephaus) have to look wether this stems from here or https://github.com/squeak-smalltalk/squeak-app .\nOn macOS we have rpath issues that can come out like this and might be sensitive to be moved around…\n\nHere we see that the lib is added in configure:\nhttps://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/483526374#L1071\n\nAnd here we see that the squeak _binary_ is actually linked agains libpulse-simple:\n\nhttps://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/483526374#L3455\n\nThat is ok, since the dynloaded vm-sound-pulse can lookup in its loading process, which (being squeak being linked against libpulse-simple) should have the syms.\n\nAccordingly we see the plugin not begin linked against the lib:\nhttps://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/483526374#L4111\n\nThe rest is libtool chitter-chat that i cannot understand so quick…\n\n\u003e \n\u003e \u003e All of the respecitve sound plugins are optional, and absence is expected.\n\u003e \u003e\n\u003e \n\u003e Not for a build slave, surely. Shouldn't;t the build slave build all\n\u003e possible variants? If we're providing prebuilt VMs (for good reason) then\n\u003e we have to build what we support. Things like vm-display-fbdev may be\n\u003e something we can not support and leave to individuals to build if they want\n\u003e to. But vm-sound-pulse is not in that category.\n\nDid not say that. all the vm-*-* plugins seem to care for optionality. and IIRC it is nigh impossible to build for some combinations to be built with the same package set (NAS + OSS or some OSS/ALSA combos, but my memory is foggy there).\n\nIn fact, I tried to make sure as many as possible plugins to be buildable on Travis: \nhttps://github.com/OpenSmalltalk/opensmalltalk-vm/commit/698661a94af388b06b98b11d9d2a4216b36acdb5#diff-852da479861e8f719725fb8018cc1699R5\nhttps://github.com/OpenSmalltalk/opensmalltalk-vm/commit/989f7e06d041c145863dd2d648e196a3169947d4#diff-eaec0bdb68c02946e8335228ebbbbad9\n\nhttps://github.com/OpenSmalltalk/opensmalltalk-vm/commit/75f59c456f79fb4f7ca64bc1d2504d3f9626e305\nhttps://github.com/OpenSmalltalk/opensmalltalk-vm/commit/ca75c6bfd0903f25917b8b88aca63e574dd39a1f\nhttps://github.com/OpenSmalltalk/opensmalltalk-vm/commit/a82b9cfdcf2f8d5396fa2f6950a06f35c374098d\nhttps://github.com/OpenSmalltalk/opensmalltalk-vm/commit/3a8a3c4bb7033c0e19c714ca4d90b0cc8f9f2565\n\nPoint is: yes, I really tried to make sure the build slave builds everything necessary.\n\n\n-t\n\n\n\u003e \n\u003e _,,,^..^,,,_\n\u003e best, Eliot\n\u003e —\n\u003e You are receiving this because you were mentioned.\n\u003e Reply to this email directly, view it on GitHub, or mute the thread.\n\u003e \n\n"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/360#issuecomment-457768165"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/360#issuecomment-457768165",
"url": "https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/360#issuecomment-457768165",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>