Minetest logo

IRC log for #minetest-dev, 2018-07-28

| Channels | #minetest-dev index | Today | | Google Search | Plaintext

All times shown according to UTC.

Time Nick Message
01:08 paramat joined #minetest-dev
01:43 ANAND Addressed the comments in #7588, anything else left to fix? :)
01:43 ShadowBot https://github.com/minetest/minetest/issues/7588 -- Improve formatting of Player HUD flags documentation by ClobberXD
01:50 twoelk left #minetest-dev
01:53 Corvus_ joined #minetest-dev
01:57 paramat will look soon
02:04 ANAND Thank you
02:14 Tmanyo joined #minetest-dev
02:16 ssieb joined #minetest-dev
03:23 paramat joined #minetest-dev
05:07 dmurph joined #minetest-dev
05:24 reductum joined #minetest-dev
08:08 reductum joined #minetest-dev
08:19 Gael-de-Sailly joined #minetest-dev
08:26 Krock joined #minetest-dev
09:13 snowmondayhappy joined #minetest-dev
09:20 Krock Okay, the build & unittest succeed now in #7569 . I'm however afraid that rounding errors might cause the unittest to fail on other systems
09:20 ShadowBot https://github.com/minetest/minetest/issues/7569 -- Send custom floats (log2) by SmallJoker
09:20 Krock fighting against noise, basically
09:33 Fixer joined #minetest-dev
09:50 snowmondayhappy joined #minetest-dev
09:52 nerzhul Krock, nice job !
09:53 Krock thanks. But it needs testing :3
09:54 ANAND joined #minetest-dev
10:35 Darcidride joined #minetest-dev
10:44 Krock Merging #7589 and #7351 in 10 minutes
10:44 ShadowBot https://github.com/minetest/minetest/issues/7589 -- Log server shutdown using actionstream by ClobberXD
10:44 ShadowBot https://github.com/minetest/minetest/issues/7351 -- Allow enter to select items from combobox's list. by basicer
10:51 ANAND Is a LATEST_PROTOCOL_VERSION bump required in #7557?
10:51 ShadowBot https://github.com/minetest/minetest/issues/7557 -- Expose player FOV to Lua API by ClobberXD
10:51 ANAND The custom floats PR bumps it from 36 to 37
10:52 ANAND When is the proto version supposed to be bumped?
10:52 ANAND Min proto version is bumped everytime backwards compat is dropped... right?
10:54 Krock well, basically I only did it because http://irc.minetest.net/minetest-dev/2018-07-25#i_5366802
10:54 Krock but it's 5.0.0-dev and the players yet don't use it on the public servers
10:55 Krock so the protocol bump is just because it would mess up too much when connecting to an old server
10:57 Krock for your case, a bump is not required because you don't need to check whether the client supports that feature or not
10:57 Krock that's in particular for backwards compatibility
10:57 Krock merging..
10:58 sfan5 ANAND: because 5.0 is a breaking release anyway we don't need to bump on every small change
10:58 sfan5 but for large changes it'd be nice to adhere to the bumping rule even if not technically required
11:06 Krock Any comments for #7568?
11:06 ShadowBot https://github.com/minetest/minetest/issues/7568 -- Check node updates whether the blocks are known by SmallJoker
11:08 sfan5 looked at it before and didn't see anything wrong
11:16 Krock a good sign :) thanks
11:21 DI3HARD139 joined #minetest-dev
11:30 Wuzzy joined #minetest-dev
11:44 ANAND Ah I see. Thanks for the clarification, Krock and sfan5 :)
11:54 nerzhul Krock if i remember we updated the MIN version on this release, your PR can increase it another time
12:50 ensonic joined #minetest-dev
13:05 Taoki joined #minetest-dev
13:44 Taoki joined #minetest-dev
16:25 xerox123_ joined #minetest-dev
16:25 xerox123 joined #minetest-dev
17:03 rubenwardy What should ContentDB use to differentiate Minetest versions internally?
17:03 rubenwardy Protocol version(?)
17:04 rubenwardy basically, you shouldn't see packages which don't support your version
17:04 rubenwardy and you shouldn't download releases which don't support your version
17:06 sfan5 protocol version is probably the best indicator you have
17:06 sfan5 you can ask users to enter which MT versions are supported and "convert" that to protocol version ranges
17:07 rubenwardy yes
17:07 rubenwardy that's my plan
17:19 rubenwardy sfan5: #7593
17:19 ShadowBot https://github.com/minetest/minetest/issues/7593 -- Add protocol_version and supported package types to ContentDB urls by rubenwardy
17:19 rubenwardy github's split view is acting very strange for me
17:20 rubenwardy unified view too
17:20 rubenwardy probably my plugin
17:21 sfan5 why does the download url need a proto version
17:21 sfan5 looks good to me either way
17:22 rubenwardy because it should get the latest release which supports their version
17:24 rubenwardy I guess it should probably return the release ID in the request, then request a specific release in the download
17:25 sfan5 yeah that would be cleaner
17:26 sfan5 also identifying a download by author sounds terrible, what if a mod changes its owner?
17:27 rubenwardy I'll be using package ID soon for storing in mod.conf etc
17:27 rubenwardy for that exact reason
17:29 Darcidride joined #minetest-dev
17:33 rubenwardy updated #7593 to do that, sfan5
17:33 ShadowBot https://github.com/minetest/minetest/issues/7593 -- Add protocol_version and supported package types to ContentDB urls by rubenwardy
17:33 rubenwardy there's still a question on the URLs containing author
17:33 rubenwardy I think it's a good idea for when people link to packages
17:34 sfan5 lgtm
17:34 rubenwardy however it does cause issues if the author of a package happens to be changes after a user downloaded the list
17:34 rubenwardy I'll wait for CI then merge
17:34 rubenwardy and solve the package ID issue later
17:35 Krock type=mod&type=game&type=txp
17:35 rubenwardy yes
17:35 Krock does [your server language] allow reading them at the same time?
17:36 rubenwardy yes
17:36 Krock nice
17:36 rubenwardy !title https://content.minetest.net/packages/?type=mod&type=game
17:36 ShadowBot rubenwardy: Mods, Games - Content DB
17:36 rubenwardy also, Flask :)
17:37 Darcidride joined #minetest-dev
17:39 Darcidride joined #minetest-dev
17:40 rubenwardy Flask is <3
17:42 Darcidride joined #minetest-dev
17:45 Darcidride joined #minetest-dev
17:48 Darcidride joined #minetest-dev
17:50 Darcidride joined #minetest-dev
17:52 Darcidride joined #minetest-dev
17:55 Darcidride joined #minetest-dev
17:56 Cornelia joined #minetest-dev
18:16 rubenwardy WTF
18:16 rubenwardy lint is suggesting mixed indentation
18:16 rubenwardy https://travis-ci.org/minetest/minetest/jobs/409339625
18:16 rubenwardy doesn't even line up
18:17 rubenwardy wait, it does
18:17 rubenwardy but still
18:47 ensonic joined #minetest-dev
19:48 Fixer joined #minetest-dev
19:57 proller joined #minetest-dev
20:14 AndroBuilder joined #minetest-dev
21:09 Foz joined #minetest-dev
21:54 rubenwardy will a memory limited process have a lower CPU %?
21:58 sfan5 what
21:59 rubenwardy from my understanding, unless the page is stored in storage the process won't become blocked
21:59 rubenwardy so the answer will be no
22:00 rubenwardy s/limited/bound/
22:49 rubenwardy merging #7593 in 10 minutes
22:49 ShadowBot https://github.com/minetest/minetest/issues/7593 -- Add protocol_version and supported package types to ContentDB urls by rubenwardy
22:52 paramat joined #minetest-dev
23:20 Wuzzy joined #minetest-dev
23:25 rubenwardy merging...

| Channels | #minetest-dev index | Today | | Google Search | Plaintext