Teeworlds - bridge

I finished deploying to windows32 and 64 yesterday ^.^

I love it ❤ great job @heinrich5991 for adding CI/CD

What do you mean?

continues integration/development

08:35

@fokkonaut stuff thats automatically checks if the build fails for example

Thats not what I asked for

08:36

I asked for the win32/64 thin

08:36

g

@ChillerDragon continous integration/deployment

09:06

stuff that automatically checks if the build fails AND automatically deploys the result (if success)

09:06

for example automatic releases on GitHub for windows when you push on the master

lol Assa? xd

what?

why do you explain to me what i already explained and nobody asked for

(no offense just lul)

10:32

oh wait fokko was asking for the win stuff thats probably related to the cd part

10:32

xd

no, you said something wrong thats why he said it correct

11:39

nobo dragon

i wouldnt consider explaining half of the thing as false information and rather as incomplete. Thats what i also marked with "for example" that i covered a example of the whole ci/cd. (sorri for this useless respond but i really wanted to say that :D)

warning: comparison is always true due to limited range of data type [-Wtype-limits]
m_pTiles[Index].m_Index <= NUM_INDICES-1

Makes no sense, NUM_INDICES is defined as 256, so it checks for <= 255, which should work for an unsigned char

13:45

Help :D

I cannot use std::stof, does someone know how I get it to work? Cant compile it on 0.7

14:19

Btw, I have stored NUM_INDICES-1 in another value now and that works as a workaround..., but I need std::stof, but cant get it

Use functions from the baselib in teeworlds for string manipulation

Which one can convert strings to floats?

14:30

lol got it

warning: comparison is always true due to limited range of data type [-Wtype-limits]

14:35

the message is explicit, the condition is always true

I know, but it shouldnt say that message

then, the test is probably useless

14:36

it is a warning

the warning doesnt appear in 0.6, but nvm. I made a new variable int Value = NUM_INDICES-1, which I use at this position. The warning does appear anymore

this kind of comparison may occur because of some error in the source code

14:38

the compiler raises it as a warning so you can check about it

14:39

NUM_INDICES might be lower in 0.6, or -Wtype-limits was used, or m_Index was not an unsigned char

No, it was the same

14:40

NUM_INDICES is from me

14:40

And the warning was on, and m_Index was an unsinged char aswell

I think you can ignore it unless NUM_INDICES get higher than 256

it wont

If I use "Hide bots" in the browser, bots from my server are still shown. Ive applied the PLAYERFLAG_BOT to them, and ingame they also have the bot skin. Whats wrong

u set the bot flag in the server info?

15:58

https://github.com/teeworlds/teeworlds/blob/master/src/engine/server/server.cpp#L1154

teeworlds/teeworlds

A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.

thanks.

Oy, why dont we have the (%d) system for names again? Also for double names

it's too much a hassle to clearly seperate similar ones

there is an official databse for similar characters

17:42

afaik DDNet uses that too

but it's not just about similar characters, names can still be similar none the less

17:58

best option is to enable id's

17:59

Dune: what about this one https://github.com/teeworlds/teeworlds/pull/2152 ?

Add greensward skin by Whis by ChillerDragon · Pull Request #2152...

Using this skin since 6 years. And can highly recommend it. Created by Whis and part of the DDNet 0.6 client. Whis gave me full permission to contribute this skin to the official client.

Oy: imho adding skins is not trivial at all and we shouldn't randomly add one skin in a minor release

18:17

Maybe it is okay to add skins in a minor release, but then it's gotta be a big update to be worthwhile the trouble

but if we do it with a major update it will take a long while

Imho the one thing that distinguishes standard skins over custom skins is that standard skins can be seen by everyone

since we want to wait with the major ones

Yes that is true

so we don'T scare of mod creators

18:18

but players are supposed to update their clients, then they would see the skins

18:19

most of them

There are some changes we can't squeeze because of that yeah

18:19

Hmm, well the way I see it is that minor releases should be optional but I may be mistaken on that

@Sonix @heinrich5991

in the past we often had security issues, where people should update

At least I would like to see a bigger update if it does happen, to justify the trouble :/

18:20

Yeah true

but if it's just a cosmetic one than there's no urge to do it yeah

And we do have the version srv

new version would have the ingame browser, maybe that helps people to update

We have some time before 0.7.4 anyway... not that much changed since last release

18:23

Yeah gotta decide what to do

already 129 commits since last release :P

I would like to get the editor in but @LordSk 🦋 is very busy

18:24

And gotta design the quads system

Merge the skin, Oy

yeah, that editor would be nice

Which editor? o.o

secret new one :P

shut up :OOOOOOOO

18:25

Any teasers? :p

@Sonix did a lot of fantastic skins

18:30

LordSk says hi

hi o/

18:37

LordSk already did most of the new editor

18:37

rest should be doable

18:37

ok to post the teaser video here?

Yeah np Oy

fokkonaut https://www.youtube.com/watch?v=sCs3a3Uz8cw

LordSk

Teeworlds: brush automap

Looks good, but why would you place the layers on the right?

Sonix asked the same thing :D

Left is better

Why does photoshop have it on the right then :p

inkscape too

18:54

probably a matter of personal taste

editor looks awesome.

set left: false

https://github.com/teeworlds/teeworlds/issues/2193

Add new skins · Issue #2193 · teeworlds/teeworlds

#2152 ...

Cool. :)

how many skins are we planning to add?

pro tip: add heart markings: one big heart and maybe 3 smaller ones - young girls like hearts

Now that we have such a nice skin system we need more skins and decorations, yes.

20:28

Oy, are you there?

kinda

Cool, this is the right way or am I retarded?

20:32

https://github.com/fokkonaut/teeworlds/commit/f1cb05b6ad427eb144767ae72e407f97ecb99e88

Add command to set fake name and clan · fokkonaut/teeworlds@f1cb05b

20:33

And there is really no way to assign the info to the own tee? :

20:33

:D*

no, not possible

20:36

looks correct

Cool

https://github.com/fokkonaut/teeworlds/commit/f1cb05b6ad427eb144767ae72e407f97ecb99e88#diff-7be5f6a31ba927b77856236df2b1b09bR276 max_clan_length

Add command to set fake name and clan · fokkonaut/teeworlds@f1cb05b

I mean why got this possibility removed?

20:39

Good one, thanks!

what possibility?

change name

20:48

Btw, can you try to build the repo on Windows with MinGW? I get weird undefined references, have a possible fix but only can test it tomorrow.

Yes you can

20:49

But mingw is messy and difficult

With VS and on Linux it works

Yes

I dont know what caused it, but it wont link due to that

20:51

I will try the possibls fix tomorrow, hoping it works

20:55

@Dune Are you on Windows?

Sometimes

If yes, can you check whether this happens with unmodified Teeworlds too?

No and I don't use mingw anyway

20:57

Because configuration is difficult

Okay, will just try that tomorrow

builds with mingw without a problem

Are you sure you checked out to the right branch?

just built master

need to build F-DDrace

Oh you are on mingw?

have both vs and mingw

Wouldn't it be better to build teeworlds with mingw to avoid msvc redistributable dependencies

21:02

Or is it built in some lib

Oy, can you also check the branch?

yeah, it's the freetype lib

21:03

took the executable provided by the freetype guys

21:03

have to build it ourself somehow

21:03

fokkonaut what branch?

did you build my repo or tw master?

tw master

Oh, okay. Could you try to build my repo (branch)?

works

writing deployment files feels like writing css

well it's scripting

Oy, did you check out to F-DDrace?

@Dune if I add an PR for an automated test-build, would you prefer to do them with tags or on every change in the master?

21:34

And should a new release be added or the "test" release be overwritten every time?

@Assa Should probably be for every release, and overwrite the previous

so for every commit - overwriting - okay ^.^

fokkonaut yeah

Lol, then I have absolutely no clue what happened for me

21:48

Thanks

getting a namespace conflict on windows with IStorage, ... https://docs.microsoft.com/en-us/windows/win32/api/objidl/nn-objidl-istorage

IStorage (objidl.h)

The IStorage interface supports the creation and management of structured storage objects.