Time  Nick      Message
12:40 Fixer     T4im: you here?
13:10 Fixer     T4im: https://github.com/minetest/minetest/issues/4348
14:02 T4im      yea, thanks
15:42 est31     as people seem to insist, I'll push this in 10 minutes: https://github.com/est31/minetest/commit/f833159a68f2ff605224ed7a5ac956e5a0ca42d8
15:50 Amaz      Is there any chance this bugfix could be merged soonish? https://github.com/minetest/minetest/pull/4128
15:53 nrzkt     est31, lgtm
15:54 est31     fine pushing then
18:18 OldCoder  For a month or two, I've observed hard lockups during server announce to list. Is this part of a new feature?
19:36 nrzkt     please note android apk is now available on github => https://github.com/minetest/minetest/releases
19:36 nrzkt     i'm late i know, i have some build problems :) and it's also published on play store, it will be available in 24-48h
19:37 nrzkt     666 issues, it's evil
19:38 paramat   i seem to know the cause of game#1109 now https://github.com/minetest/minetest_game/issues/1109#issuecomment-234795907
19:38 ShadowBot https://github.com/minetest/minetest_game/issues/1109 -- Slabs get a static facedir
19:44 paramat   nore sfan5 sofar please can we merge #965 now? further work is needed on the mod but this PR is very old and the author a bit fed up
19:44 ShadowBot https://github.com/minetest/minetest/issues/965 -- Portability fixes for OpenBSD (and possibly NetBSD and others). by Warr1024
19:44 paramat   game#965
19:44 ShadowBot https://github.com/minetest/minetest_game/issues/965 -- Ability to Disable bones, drop items or keep inventory. by tenplus1
19:48 sfan5     does the code work?
19:50 paramat   it works yes, tested on xanadu server, but we need to do further work on it, as you can see from the recent comments
19:52 T4im      hm, seems to work for me though, everamzah: did you get the latest commit of him too? the one with the additional check?
19:53 T4im      I could imagine, that an empty string might not be well handled (it only tests for nil)
19:53 T4im      for the setting/mode
19:56 tenplus1  hi folks
19:56 T4im      hey :)
19:57 est31     hello
19:59 tenplus1  on line 242 of pull code you guys set time when in protected to  (share_bones_time - share_bones_time_early) ??? should it just be share_bones_time_early which is 1/4 of original time ?
19:59 paramat   nore sfan5 , sofar is inactive due to RL stuff so we still need your help
20:02 paramat   hopefully ruben wardy might be happy to join as a dev later
20:03 tenplus1  how about we do this:  if bones and cannot replace, y+1, if bones and cannot replace, change to drop, that way it forces bones unless it cant officially place item die to obstruction then it drops anyhoo ?
20:04 tenplus1  that way we all win
20:07 T4im      I think it works fine now, actually; at least where it sets bones; I'm no wondering about why everamzah meant the node time would be reduced, were there any chances to that end?
20:07 tenplus1  game#965 changed
20:07 ShadowBot https://github.com/minetest/minetest_game/issues/965 -- Ability to Disable bones, drop items or keep inventory. by tenplus1
20:08 T4im      eh?
20:11 paramat   i need to get my head around bones (heh) i have misunderstood
20:12 tenplus1  when set to "bones" it will check if it can place bones, if it cannot due do obstruction it increases Y + 1
20:12 tenplus1  if it still cannot place bones due to obstruction then it forces "drop" as mode
20:12 tenplus1  hows that ????
20:12 tenplus1  and of course if "drop" is already set it's no change
20:13 tenplus1  it only checks protected areas when deciding TIME for when bones turns old
20:13 paramat   ok will consider
20:14 tenplus1  testing now
20:14 T4im      I'm not quite seeing what the last commit even changed, tbh, in both cases it dropped on second obstruction, in both cases it dropped on drop, in both cases it tried to place the bones y+1 before dropping
20:15 T4im      or am I missing something here?
20:15 T4im      :D
20:15 tenplus1  with latest pull, setting to "bones" will do it's best to place a bones block and only IF it's obstructed at pos and+1 pos then it'll drop items isntead
20:18 T4im      yea, but it did that before, too, from what I understand
20:18 tenplus1  code change works fine
20:18 tenplus1  it's now back to DEFAULT behavious
20:18 tenplus1  *behaviour
20:19 T4im      however now, if you set the setting to anything but drop or bones or keep, it will not do any may_replace checks anymore
20:19 T4im      before it did at least one check and dropped
20:19 tenplus1  may_replace only checks for defined node and obstruction... NOT protection
20:19 T4im      it does for buildable_to nodes
20:19 tenplus1  it only checks for protection when a bone is placed and it has to determine bone time
20:21 tenplus1  so when in "bones" mode, if POS is obstructed it has Y+1, check again, if POS is obstructed then it changes mode to "drop"...  if NOT obstructed then places bones and checks for protected area, if protected bones_time is set to early
20:21 tenplus1  that's it... default behavious AND drops all in 1 mod, both work, both tested
20:21 T4im      have you tested setting bones_mode to "brones"?
20:21 tenplus1  if set to "keep" or in creative mode then items do not drop
20:22 tenplus1  if it's set to something other than "keep", "drop" or "bones" then it defaults to bones
20:22 T4im      no
20:22 T4im      only if nil
20:22 paramat   T4im i was wrong, current behaviour is fine
20:22 tenplus1  the IF statements take care of that
20:23 tenplus1  if spelled incorrectly it'll default to bones EVEN WHEN NOT NIL
20:23 T4im      yes, if spelled correctly
20:23 T4im      then it behaves like the previous commit of yours
20:24 tenplus1  if the user/admin enters an incorrect setting then THEY should fix the setting
20:24 tenplus1  mods only work when settings are used properly...
20:24 T4im      I'm just pointing out, that your previous commit was fine
20:24 T4im      the last one was kinda unnecessary and made typos destructive
20:24 T4im      that's all
20:26 nolsen    It seems in Ubuntu 16.10, libpng12-dev is actually replaced with libpng12-0, should that be changed?
20:26 nolsen    Since libpng12-0 is available on Ubuntu 16.04
20:26 nolsen    in the readme.
20:27 nolsen    (Running a bleeding edge copy of Ubuntu 16.10, bleeding edge is love, bleeding edge is life)
20:27 tenplus1  fixed...  typos wont affect default
20:28 tenplus1  nite folks...
20:29 paramat   tenplus latest version of the PR seems potentially good
20:38 paramat   +1
21:52 nolsen    Is this normal? https://nolsen.xyz/paste/?f0bfc4ccedae4a64#fxDFHOJeH2MjO9NluShbXMjRV3CZtWGqop1v0khS4QU=
23:13 paramat   game#1208 is seemingly complete
23:13 ShadowBot https://github.com/minetest/minetest_game/issues/1208 -- Default: Prevent placing sapling if grown tree intersects protection by paramat
23:15 nolsen    ERROR[CurlFetch]: servers.minetest.net/announce not found (HTTP response code said error) (response code 500)
23:15 nolsen    Um, is that supposed to happen?
23:18 nolsen    Err, nevermind, the fix was removing the IPv6 DNS entry.