Time |
Nick |
Message |
01:47 |
|
v-rob joined #minetest-dev |
02:39 |
|
v-rob joined #minetest-dev |
03:17 |
|
[MTMatrix] joined #minetest-dev |
03:49 |
|
vampirefrog joined #minetest-dev |
04:00 |
|
MTDiscord joined #minetest-dev |
04:23 |
|
Pexin joined #minetest-dev |
05:03 |
|
v-rob joined #minetest-dev |
06:19 |
|
[MTMatrix] joined #minetest-dev |
11:43 |
|
Noisytoot joined #minetest-dev |
12:14 |
|
Lupercus joined #minetest-dev |
12:29 |
|
Noisytoot joined #minetest-dev |
12:56 |
|
Noisytoot joined #minetest-dev |
13:39 |
MTDiscord |
<herowl> Why can't we have nice things? (like c++20) |
13:43 |
celeron55_ |
are we tracking the status of C++20 on our target platforms at the moment? |
13:43 |
|
grorp joined #minetest-dev |
13:43 |
grorp |
merging #14472 in 15 min (thanks sfan5) |
13:43 |
ShadowBot |
https://github.com/minetest/minetest/issues/14472 -- TouchScreenGUI dehardcoding refactor by grorp |
13:43 |
sfan5 |
i don't think so |
13:43 |
sfan5 |
merging #14546 in 15m after grorp's merge |
13:43 |
ShadowBot |
https://github.com/minetest/minetest/issues/14546 -- Fix missing DLLs in Windows build by sfan5 |
13:46 |
celeron55_ |
i think there's starting to be a high chance it's supported on each platform |
13:47 |
celeron55_ |
one good thing is that centos was finally discontinued. it tended to be worse than debian |
13:48 |
celeron55_ |
do we consider g++ 12 to have enough c++20 support? |
13:48 |
sfan5 |
i think we first have to define what we event want c++20 for |
13:48 |
sfan5 |
spans? |
13:49 |
celeron55_ |
@herowl which features would you like to use from c++20? |
13:52 |
celeron55_ |
debian stable seems to have gcc 12 and glibc 6, if i'm not mistaken |
13:52 |
sfan5 |
wasn't there once some PR that formalized the "which platforms do we support" thing? |
13:52 |
celeron55_ |
span is in glibc 7, altough i'm 55% certain i'm looking this up wrong |
13:53 |
sfan5 |
that should be libstdc++, not glibc |
13:53 |
celeron55_ |
wait |
13:53 |
celeron55_ |
this was the llvm one, yes |
13:54 |
sfan5 |
[2023-06-11 21:23:56] <sfan5> anyway for estimating which compilers or libraries we can require I have suggested in the past to check the oldest still-supported Ubuntu, Debian and maybe Fedora |
13:54 |
celeron55_ |
https://gcc.gnu.org/projects/cxx-status.html#cxx20 |
13:54 |
MTDiscord |
<herowl> At this point I wanted to use https://en.cppreference.com/w/cpp/string/basic_string/ends_with |
13:54 |
celeron55_ |
https://gcc.gnu.org/onlinedocs/libstdc++/manual/status.html#status.iso.2020 |
13:55 |
sfan5 |
we have str_ends_with for that in util/string.h |
13:57 |
celeron55_ |
ok so anyway debian stable is at 12 for g++ and libstdc++, which makes sense (i'm sure differing versions don't go well together...) |
14:00 |
celeron55_ |
it's difficult to figure out the details (like ends_with) but it seems to me debian is probably good here |
14:00 |
celeron55_ |
ubuntu 22.04 LTS is at 11, which might be too old |
14:02 |
celeron55_ |
and what's worse, the older ubuntu 20.04 LTS has official support until 2025. it uses gcc 9 |
14:03 |
celeron55_ |
fedora has a tight release schedule, i don't think it will ever be the bottleneck for us |
14:03 |
nrz |
celeron55_ better to target C++17 first which should be fine, C++20 is not widely implemented |
14:04 |
nrz |
C++17 permit to remove nearly all our porting around filesys |
14:05 |
celeron55_ |
not everything out of c++17 is supported by gcc 9, but most things are |
14:05 |
celeron55_ |
it would probably be doable now |
14:05 |
nrz |
gcc 9 is the one in debian 11 right ? |
14:05 |
celeron55_ |
actually, gcc 9 does support everything in c++17, but the standard library doesn't have everything |
14:06 |
celeron55_ |
i'm looking at debian bookworm which is the stable now |
14:06 |
celeron55_ |
whatever the number for that is. debian doesn't really show the number anywhere |
14:07 |
celeron55_ |
and this debian has gcc 12 |
14:07 |
celeron55_ |
it's not the bottleneck |
14:07 |
celeron55_ |
ubuntu LTS is the bottleneck |
14:07 |
nrz |
if the issue is glibc, we have another problem then 😄 |
14:07 |
celeron55_ |
ubuntu LTS has gcc 9 |
14:10 |
celeron55_ |
anyway, i agree with nrz about c++17. support for it is ok now. just make sure our CI does a build on ubuntu 20.04 LTS as that's the worst target platform |
14:13 |
celeron55_ |
and yes, let me rephrase that: the "Worst C++ Target Platform of 2024" prize goes to Ubntu 20.04 LTS. go ahead and give them an applause |
14:13 |
celeron55_ |
+u |
14:18 |
nrz |
it's LTS - 2 this month , 24.04 will replace 22.04 |
14:19 |
celeron55_ |
doesn't change the fact that it's still supported by ubuntu until 2025 |
14:20 |
nrz |
right 🙂 |
14:55 |
sfan5 |
we're already on c++17 :) |
15:43 |
pgimeno |
is Debian LTS unsupported by Minetest? |
15:44 |
pgimeno |
Debian LTS has GCC 8.3 |
15:47 |
pgimeno |
support ends in June though |
15:48 |
sfan5 |
it probably works on there |
15:49 |
nrz |
sfan5, oh amazing, missed it |
16:11 |
sfan5 |
https://www.minetest.net/downloads/ it's funny how nobody ever updated the BSD instructions to tell people to *not* install MTG |
16:15 |
MTDiscord |
<zmv7> PR welcome? 😏 |
16:17 |
sfan5 |
always |
16:17 |
sfan5 |
https://github.com/minetest/minetest.github.io |
16:51 |
rubenwardy |
Why is Ubuntu 24.04 still shipping with 5.6? Who do we complain to about this https://packages.ubuntu.com/search?keywords=minetest |
16:52 |
sfan5 |
https://tracker.debian.org/pkg/minetest -> maintainer: Debian Games Team |
16:52 |
rubenwardy |
oh right |
16:53 |
rubenwardy |
5.8.0 is still in unstable, is it likely to miss 24.04 then |
16:53 |
sfan5 |
s/likely/guaranteed/ |
16:53 |
rubenwardy |
so Minetest is going to be 3.5 years outdated by the time we get to the next lts :/ |
16:54 |
rubenwardy |
this will really drag out any renaming attempts |
16:54 |
ROllerozxa |
has debian debundled MTG from their base minetest 5.8.0 package? |
16:55 |
rubenwardy |
yes |
16:55 |
rubenwardy |
and disabled the update checker |
16:57 |
rubenwardy |
https://gist.github.com/rubenwardy/eb481b51a5e0574e5219b6967752bdb0 |
16:58 |
ROllerozxa |
huh. I'm surprised they didn't keep MTG just to be petty |
16:59 |
rubenwardy |
what I want from a distro is a stable base/os and up to date applications. Having stale applications, especially when they're networked, is quite annoying |
16:59 |
rubenwardy |
I suppose a 'stable' distro and flatpak is the most feasible way to achieve tihs |
17:00 |
MTDiscord |
<herowl> rubenwardy: I implemented most of your comments in #14543 |
17:00 |
ShadowBot |
https://github.com/minetest/minetest/issues/14543 -- WIP: Add gameid aliases by nauta-turbidus |
17:00 |
MTDiscord |
<herowl> I have problems with the updater though |
17:00 |
rubenwardy |
nice! |
17:00 |
ROllerozxa |
actually I checked the file list of the version of minetest-data in debian sid (5.8.0+dfsg+~1.9.0mt13-1) and there's both devtest and MTG in here |
17:00 |
rubenwardy |
eek |
17:02 |
ROllerozxa |
> Install games directory manually, as upstream's make install no longer does it. |
17:02 |
ROllerozxa |
Debian™ |
17:02 |
rubenwardy |
oh I thought minetest-data is deprecated? |
17:02 |
ROllerozxa |
minetest-data is where the common data is stored used for both the client and server package |
17:02 |
rubenwardy |
grr |
17:04 |
sfan5 |
projects usually deal with this by rejecting bug reports from debian users, because their application could be patched in many surprising ways :) |
17:06 |
|
Wuzzy joined #minetest-dev |
17:07 |
MTDiscord |
<herowl> rubenwardy: I am not really sure how updater should deal with a rename. Should it just go through the list of aliases and remove any game folders going by those names? What if somebody manually created a folder named like that? |
17:07 |
rubenwardy |
You don't need to read the aliases, just look at package.path |
17:07 |
rubenwardy |
It will point to the old name |
17:08 |
rubenwardy |
What Debian should do is move MTG to its own package, and at most Recommend it |
17:15 |
ROllerozxa |
yeah |
17:16 |
ROllerozxa |
I mean they could package however many games for debian as they want, just like how they package a bunch of ancient mods |
17:16 |
sfan5 |
there we go: #14556 <- feedback wanted |
17:16 |
ShadowBot |
https://github.com/minetest/minetest/issues/14556 -- Formalize OS support policy by sfan5 |
17:18 |
sfan5 |
I just realized we should exclude debian oldoldstable because that doesn't work even now |
17:31 |
MTDiscord |
<josiah_wi> Updating CMake to a newer version for development is very straightforward. |
17:32 |
sfan5 |
I know you can download a binary from the cmake website, are you getting at that? |
17:32 |
|
v-rob joined #minetest-dev |
17:32 |
MTDiscord |
<josiah_wi> I think there's a script that updates it for you, but I have not personally used it since last summer I think, so I don't remember the details. |
17:34 |
MTDiscord |
<josiah_wi> I think it would interfere with any installed system version, so it's only useful on a development-specific environment. |
17:34 |
MTDiscord |
<josiah_wi> Depending on your workflow I suppose that would be frustrating to have to update. |
17:35 |
MTDiscord |
<josiah_wi> Not on the same level as compiling a newer version of GCC though. |
17:38 |
sfan5 |
my opinion on downloading cmake binaries: sure it works but I feel it's bad practice. a mere build system is not something I'd want to download separately |
17:40 |
* v-rob |
wants C++20 modules and C++23 import std; but only MSVC supports them fully |
17:41 |
v-rob |
It's a strange day, when Microsoft supports the standards before GCC or Clang do |
18:19 |
sfan5 |
also could someone take a look at #14534? It's not complex but also not simple enough that I want to merge it with 1 approval |
18:19 |
ShadowBot |
https://github.com/minetest/minetest/issues/14534 -- [no sq] Profiler improvement + an optimization by sfan5 |
18:28 |
MTDiscord |
<herowl> OK, I can't figure out how do I check if it should be renamed? Because it looks like after a rename update installs to the old folder, right? |
18:31 |
MTDiscord |
<herowl> @rubenwardy |
18:33 |
rubenwardy |
get the folder name and compare it with the package name (after normalising to remove _game) |
18:42 |
MTDiscord |
<herowl> rubenwardy: is there a function to rename a directory? |
18:42 |
MTDiscord |
<herowl> or do I copy and remove the original dir? |
18:43 |
MTDiscord |
<herowl> as I don't see a move function either... |
18:44 |
rubenwardy |
delete the old dir |
18:45 |
rubenwardy |
mt already deletes the old dir when updating |
18:45 |
rubenwardy |
so the behaviour should be to delete the correct old dir and install to the correct new dir |
18:49 |
MTDiscord |
<herowl> rubenwardy: ok, how do I test it? I guess we don't have such cases in CDB? |
18:51 |
rubenwardy |
In the fetch_pkgs function in contentdb.lua, you could modify `aliases` of a specific game |
18:51 |
rubenwardy |
Say you when the package.name == mineclone2, you could add package.aliases = { "foobar" }. Then locally rename the mineclone2 folder to foobar |
18:51 |
MTDiscord |
<herowl> alright |
18:52 |
rubenwardy |
* aliases = { "Wuzzy/foobar" } |
18:53 |
rubenwardy |
you'll also need to reduce `release` in the game.conf so it detects an update |
18:58 |
MTDiscord |
<herowl> works |
19:59 |
MTDiscord |
<luatic> why not use os.rename if it's just a rename? copying & deleting seems unnecessarily wasteful |
20:00 |
MTDiscord |
<herowl> Well, it deletes the old directory when updating and writes it anew anyway |
20:00 |
MTDiscord |
<herowl> so I leveraged that |
20:00 |
MTDiscord |
<herowl> (no extra copies and deletes) |
20:00 |
MTDiscord |
<luatic> ah okay |
20:01 |
MTDiscord |
<herowl> Everything is at https://github.com/minetest/minetest/pull/14543 if you want to take a look |
20:35 |
|
v-rob joined #minetest-dev |
22:14 |
|
turtleman joined #minetest-dev |
22:32 |
|
v-rob joined #minetest-dev |
22:33 |
|
panwolfram joined #minetest-dev |