Minetest logo

IRC log for #minetest-dev, 2013-09-25

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

All times shown according to UTC.

Time Nick Message
00:09 kahrl PilzAdam: to distinguish git builds, we could modify the part in CMakeLists.txt where VERSION_STRING is generated so that it runs "git describe --always --tag" and if that succeeds, it overrides VERSION_STRING
00:10 PilzAdam seems good, but it should fallback to "-dev" if it fails (i.e. it isnt a git repo)
00:11 kahrl in that case the user may have downloaded e.g. the 0.4.7 source as a zip
00:11 kahrl so -dev wouldn't be appropriate
00:12 PilzAdam comment that line out for stable releases then
00:13 kahrl I'm trying to make releasing as little painful as possible
00:14 kahrl hmm, one thing that git describe doesn't catch is uncommitted modifications
00:15 kahrl guess we'll have to live with that
00:17 kahrl oh there's --dirty
00:17 werwerwer joined #minetest-dev
00:18 kahrl so: git --git-dir="${CMAKE_CURRENT_SOURCE_DIR}" describe --always --tag --dirty
00:27 VanessaE kahrl: +1000
00:29 VanessaE it would make modders' lives easier too ("oh, what version?"  "um..0.4.7...."  "well, what's the exact text up in the upper left corner?"  "um...oh 0.4.7-ab57dc89blahblah"  "Oh, that's why, yours is too old, get 584a2d93bblahblah")
00:29 PilzAdam VanessaE, yes, that was the idea
00:30 kahrl PilzAdam: I think the idea with manually disabling the line on releases is the sanest, can't think of anything else
00:30 VanessaE PilzAdam: the last time I pitched this idea, I was turned down because it was thought that not enough people would even see it because of the origin of their builds' source code.
00:33 kahrl what should we do about VERSION_EXTRA?
00:33 PilzAdam use that instead of the other thing
00:33 kahrl what other thing?
00:33 PilzAdam the git version
00:34 kahrl why?
00:35 PilzAdam e.g. in my builds usually have a name (like "vmanip") that can be used instead of the git version
00:36 VanessaE is VERSION_EXTRA is set, use that, and if not, plug in the git commit id then - or just use both
00:38 kahrl okay
00:38 PilzAdam VERSION_EXTRA is used to describe the version; the git version has the same purpose, so it doesnt make much sense to print both at once
00:38 kahrl you lose the exact commit and dirty info that way
00:38 VanessaE sure it does - what about the aforementioned uncommitted changes?
00:39 VanessaE exactly.
00:39 PilzAdam kahrl, well, its the job of the one who sets the version_extra to take care that he knows the exact version
00:40 kahrl that's true as well
00:41 kahrl most users aren't ever going to see a version with VERSION_EXTRA set anyway
00:42 NakedFury where do you see all this info anyway?
00:42 NakedFury as a "most user"
00:43 kahrl top left of the windows
00:43 kahrl -s
00:43 kahrl or --version once it is implemented
00:44 kahrl what to do with the CPACK variables? just leave them as they are?
00:47 NakedFury add all that to About
00:49 NakedFury if it ends up being too much info add it to an About or Info tab
00:50 PilzAdam its only for dev builds, so it should be quite visible
00:50 PilzAdam and not hidden somewhere
00:59 hax404 joined #minetest-dev
01:19 salamanderrake joined #minetest-dev
01:30 Sokomine was anything changed recently (relatively) regarding moving around of inventory?
01:32 Sokomine i do have the following problem: an inventory place/slot allows take but not place. when something is taken out of that slot and placed into a slot of the players inventory that already contains a diffrent items, those are swapped...which does not work because the slot does not accept input
01:35 kahrl Sokomine: maybe related to d74c377337?
01:37 blaaaaargh joined #minetest-dev
01:37 Sokomine how do i find out what that code stands for?
01:38 kahrl git diff d74c377337^ d74c377337
01:41 kahrl Sokomine: https://github.com/minetest/minetest/issues/332
01:41 Sokomine could very well be. i'm having difficulties with my circular saw which seems to lead to item loss under some circumstances
01:42 Sokomine looks related
01:48 Neological joined #minetest-dev
02:00 AllegedlyDead joined #minetest-dev
02:09 IceCraft joined #minetest-dev
02:42 werwerwer joined #minetest-dev
02:55 ShadowNinja I need a way to get all of the lists in a InvRef, specificaly a player InvRef. Is there any way to do that currently? If not can someone add it quickly? to_table or get_listnames would be good.
03:12 Miner_48er joined #minetest-dev
03:18 kahrl I thought that was added some time ago. Maybe not
03:45 ImQ009 joined #minetest-dev
04:29 nyuszika7h joined #minetest-dev
04:37 Sokomine joined #minetest-dev
04:51 celeron55 joined #minetest-dev
05:16 ShadowNinja minetest.set_player_privs interprets false as true, you need to use nil to remove privileges.
05:16 neko259 joined #minetest-dev
05:16 ShadowNinja I may fix that tomorow if nobody else has yet.
05:44 salamanderrake joined #minetest-dev
06:57 Calinou joined #minetest-dev
07:00 darkrose joined #minetest-dev
08:02 Ritchie joined #minetest-dev
08:11 rubenwardy joined #minetest-dev
08:18 ecube_ joined #minetest-dev
08:56 darkrose joined #minetest-dev
10:57 proller joined #minetest-dev
11:34 PilzAdam joined #minetest-dev
11:44 rubenwardy joined #minetest-dev
12:01 PilzAdam kahrl, you should write "kahrl" and not "celeron55" as the copyright holder in new files
12:14 celeron55 PilzAdam: almost everyone has written my name on new files always so it's practically meaningless because no transfer of copyright ownership has really officially been made
12:15 celeron55 you can write "santa claus" there if you please; nobody can use the name there for any purpose anyway
12:15 celeron55 (except using it for contact information for asking who the actual copyright holder possibly might be)
12:16 PilzAdam its a good way to step away from the "celeron-centrtic development"
12:16 celeron55 in that case someone should look through git blames of files and correct the names where possible
12:17 ImQ009 joined #minetest-dev
12:26 lanxu heh. "moving away from celeron-centric development". Now supporting even more CPU brands, right? :>
14:27 Semilevel joined #minetest-dev
14:55 Zeitgeist_ joined #minetest-dev
14:56 Jordach joined #minetest-dev
15:08 kahrl whatever name is put there becomes meaningless after a while
15:09 kahrl I mean, look at the cpp files introduced by lua mainmenu, they all have (C) sapier but some have quite a number of edits by other people already
15:11 PilzAdam it gives the impression that many people work on Minetest instead of just one
15:11 PilzAdam (if you read the code)
15:12 kahrl well I don't really care about that
15:12 kahrl do people?
15:13 PilzAdam well, all the other arguments say that it doesnt matter
15:13 kahrl I think most would look at the authors of commits first instead of copyright lines
15:16 kahrl we could start adding a changelog with attribution at the top of every file, but that would really bloat up the commits with fluff
15:17 ShadowNinja No changelog in the repo, although one on a wiki or some such is good. (Which we already have)
15:17 PilzAdam well, I dont really care as long as celeron55 is ok when people write his name and as long as its allowed for contributors to put their name in new / mostly rewritten files
15:17 kahrl that's how it has been
15:19 celeron55 maybe the copyright notices could be changed to "Individual Minetest developers" or something; that would be truthful at least
15:19 PilzAdam I would be ok with removing all names too
15:20 kahrl yeah, me too
15:20 ShadowNinja "Minetest development team?"
15:21 celeron55 the copyright is not owned by any signle legal entity
15:21 celeron55 single*
15:22 celeron55 for example "Minetest development team" sounds like the team is some kind of a legal organization to which people transfer the copyrights and that can decide on licensing
15:24 VanessaE in American slang at least, "team" can also mean a loosely-organized group of people that merely follow a common goal.
15:24 PilzAdam is American slang used in courts now?
15:24 celeron55 8D
15:25 VanessaE actually, yes it is :P
15:25 VanessaE though the courts tend to prefer somewhat more well-defined concepts :)
15:26 celeron55 but the core thing here is that it doesn't actually matter what is written in there, so we might just as well let it be unless it's going to be changed to something more factual
15:50 Semilevel joined #minetest-dev
15:51 Calinou joined #minetest-dev
16:32 rubenwardy joined #minetest-dev
16:44 smoke_fumus joined #minetest-dev
16:48 NakedFury joined #minetest-dev
16:59 proller joined #minetest-dev
18:12 iqualfragile joined #minetest-dev
18:13 neko259 joined #minetest-dev
18:14 iqualfragile joined #minetest-dev
18:21 iqualfragile joined #minetest-dev
18:22 Miner_48er joined #minetest-dev
19:53 ShadowNinja It seems visual_size only accepts x and y values, not z. Can that be added?
20:00 salamanderrake joined #minetest-dev
20:04 salamanderrake joined #minetest-dev
20:20 hmmmm joined #minetest-dev
20:26 kaeza joined #minetest-dev
20:31 hmmmm joined #minetest-dev
20:35 caemir joined #minetest-dev
20:42 salamanderrake joined #minetest-dev
22:33 prestotron55 joined #minetest-dev
22:37 Miner_48er joined #minetest-dev
22:56 diemartin joined #minetest-dev

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