<div dir="ltr"><div dir="ltr"><br></div><div>1) don't trust documentation, trust tests (by running them)</div><div><br></div><div>2) with BR, it's easy to run a test (even locally, before a commit), but it isn't easy to know that a test exists.</div><div>In my point of view, the right place of test_prosody.py is in the directory package/prosody, not in support/testing/tests/package,</div><div>support/testing is the right place only for testing infrastructure.<br></div><div><br></div><div>François<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le jeu. 4 juin 2020 à 00:42, Romain Naour <<a href="mailto:romain.naour@gmail.com">romain.naour@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi James,<br>
<br>
Le 04/06/2020 à 00:30, James Hilliard a écrit :<br>
> On Wed, Jun 3, 2020 at 3:59 PM Romain Naour <<a href="mailto:romain.naour@gmail.com" target="_blank">romain.naour@gmail.com</a>> wrote:<br>
>><br>
>> This reverts commit fa84c176c2148a60103e850204180f86aa5baa73 that<br>
>> replace luabitop by lua_bit32 package when lua 5.1 is used.<br>
>><br>
>> Since this change the prosody test in gitlab is fail due to<br>
>> missing lua-bitops [1]:<br>
>><br>
>> Starting prosody:<br>
>> **************************<br>
>> Prosody was unable to find lua-bitops<br>
>> This package can be obtained in the following ways:<br>
>><br>
>>     Source:           <a href="http://bitop.luajit.org/" rel="noreferrer" target="_blank">http://bitop.luajit.org/</a><br>
>>     Debian/Ubuntu:    sudo apt-get install lua-bitop<br>
>>     luarocks:         luarocks install luabitop<br>
>><br>
>> WebSocket support will not be available<br>
>> More help can be found on our website, at <a href="https://prosody.im/doc/depends" rel="noreferrer" target="_blank">https://prosody.im/doc/depends</a><br>
>> **************************<br>
>><br>
>> The upstream documentation [2] is misleading (or not uptodate)<br>
>> about lua-bit32 dependency.<br>
>><br>
>> Since bitop is builtin since lua5.2, we probably need to select<br>
>> luabitop package only when lua 5.1 is used as lua interpreter.<br>
>><br>
>> Tested with run-tests:<br>
>> ./support/testing/run-tests tests.package.test_prosody.TestProsodyLua51<br>
>><br>
>> [1] <a href="https://gitlab.com/buildroot.org/buildroot/-/jobs/576271975" rel="noreferrer" target="_blank">https://gitlab.com/buildroot.org/buildroot/-/jobs/576271975</a><br>
> Isn't this job failure just due to this faulty test?<br>
> self.module_test('bit') # luabitop<br>
> Which should be replaced with:<br>
> self.module_test('bit32') # luabit32<br>
> or something along those lines?<br>
<br>
I'm not speaking about the test itself but the prosody log when it's started<br>
"Prosody was unable to find lua-bitops"<br>
<br>
Also the test is shared with luajit and the TestProsodyLuajit test is fine.<br>
<br>
Best regards,<br>
Romain<br>
<br>
>> [2] <a href="https://prosody.im/doc/depends#bitop" rel="noreferrer" target="_blank">https://prosody.im/doc/depends#bitop</a><br>
>><br>
>> Signed-off-by: Romain Naour <<a href="mailto:romain.naour@gmail.com" target="_blank">romain.naour@gmail.com</a>><br>
>> Cc: James Hilliard <<a href="mailto:james.hilliard1@gmail.com" target="_blank">james.hilliard1@gmail.com</a>><br>
>> Cc: Yann E. MORIN <<a href="mailto:yann.morin.1998@free.fr" target="_blank">yann.morin.1998@free.fr</a>><br>
>> ---<br>
>>  package/prosody/Config.in | 2 +-<br>
>>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
>><br>
>> diff --git a/package/prosody/Config.in b/package/prosody/Config.in<br>
>> index 0b01547103..c32ca20e23 100644<br>
>> --- a/package/prosody/Config.in<br>
>> +++ b/package/prosody/Config.in<br>
>> @@ -4,7 +4,7 @@ config BR2_PACKAGE_PROSODY<br>
>>         depends on BR2_PACKAGE_HAS_LUAINTERPRETER<br>
>>         depends on !BR2_PACKAGE_LUA_5_3<br>
>>         depends on !BR2_STATIC_LIBS # luaexpat, luasec, luasocket, luafilesystem<br>
>> -       select BR2_PACKAGE_LUA_BIT32 if BR2_PACKAGE_LUA_5_1 # runtime<br>
>> +       select BR2_PACKAGE_LUABITOP if !BR2_PACKAGE_LUAJIT # runtime<br>
>>         select BR2_PACKAGE_LUAEXPAT # runtime<br>
>>         select BR2_PACKAGE_LUASEC # runtime<br>
>>         select BR2_PACKAGE_LUASOCKET # runtime<br>
>> --<br>
>> 2.25.4<br>
>><br>
<br>
_______________________________________________<br>
buildroot mailing list<br>
<a href="mailto:buildroot@busybox.net" target="_blank">buildroot@busybox.net</a><br>
<a href="http://lists.busybox.net/mailman/listinfo/buildroot" rel="noreferrer" target="_blank">http://lists.busybox.net/mailman/listinfo/buildroot</a><br>
</blockquote></div></div>