DDraceNetwork - developer

meloƞ

@Ewan sry for ping but i love having these talks with ya - compiling with cmake .. and building with cmake --builld . -j wont move neccessary .dll files from \ddnet\build to \ddnet\build\debug where the compiled files and the .exe files reside - resulting in missing .dll files when launching ddnet - isnt there supposed to be a set command in the cmakelists copying those over automatically? or am i missing something for context: - simply moving ddnet.exe and the steam_api.dll (that somehow got correctly generated there) - makes it Executable and it works just fine. (edited)

are you talking about dynamic libs (compiled alongside DDNet) or shared libs (on the system)?

00:46

as far as the latter is concerned, it should typically be handled as an install step. whatever environment you’re using with mingw should be smart enough to load the dependencies without any fuss as you run it (edited)

00:47

for Windows deployment this is tricky because there’s no expectation that the libs will exist on target systems. generally when you “deploy” you should also copy the relevant DLLs over. Copying stuff like that should not be the job of the build system (at compile time) (edited)

00:48

I use this script https://gist.github.com/ewancg/de0b47f55b9894b9dc5a7b8609150186

Copy dependency DLLs of a program built with MSYS2 into current dir...

Copy dependency DLLs of a program built with MSYS2 into current directory - msys2-deploy.sh

00:48

do you really want make making unneeded copies of all of the system DLLs?

00:52

@meloƞ i forgot if i pinged you with the initial reply... forgive the dupe if so

00:53

this is also a licensing can of worms. you're essentially creating a build with all the contents of a static build, which has its own considerations (some libraries you are restricted from linking statically by license)

ChillerDragon

[ddnet/ddnet] Pull request opened: #8263 Use LineAuthor variable to cleanup chat code

Checklist

[x] Tested the change ingame
[ ] Provided screenshots if it is a visual change
[ ] Tested in combination with possibly related configuration options
[ ] Written a unit test (especially base/) or added coverage to integration test
[ ] Considered possible null pointers and out of bounds array indexing
[ ] Changed no physics that affect existing maps
[ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssan...

<ChillerDragon> That CI fail is suspicious. https://github.com/ddnet/ddnet/actions/runs/8825297278/job/24229302734?pr=8236

Add delta debug dump · ddnet/ddnet@29f6cc6

DDraceNetwork, a free cooperative platformer game. Contribute to ddnet/ddnet development by creating an account on GitHub.

01:22

<ChillerDragon> the pr added an unused method that is never called. And the CI failed in the admin integration test. Are the integration tests getting unstable?

github-merge-queue[bot]

[ddnet:master] 6 new commits

fc7f550 Update Turkish translations - GokturkTalha df3c37e Update Turkish translations - GokturkTalha 118e643 Update spanish.txt - n0Ketchp ddeec05 finish.txt - GokturkTalha 8fb0052 Merge pull request #8259 from n0Ketchp/patch-18 - def- 708a9c4 Merge pull request #8258 from GokturkTalha/patch-5 - def-

02:30

def-

[ddnet/ddnet] Pull request closed: #8258 Update Turkish translations

02:30

def-

[ddnet/ddnet] Pull request closed: #8259 Update spanish.txt

02:31

def-

[ddnet/ddnet] Pull request closed: #8257 Add Azerbaijanese translation

02:31

github-merge-queue[bot]

[ddnet:master] 5 new commits

13452b3 Add Azerbaijanese translation - GokturkTalha 54024cd Add Azerbaijanese translation - GokturkTalha 8ac9ea4 Add Azerbaijanese translation - GokturkTalha 712b50a final - GokturkTalha 3825eb7 Merge pull request #8257 from GokturkTalha/patch-4 - def-

Melon

Ewan

are you talking about dynamic libs (compiled alongside DDNet) or shared libs (on the system)?

Solely talking about dynamic libs (sqlite, steamapi, avformat, sdl2) - i'll Check If i can modify my cmakelists to do that for me. Thanks ❤️

Vappy

Melon

Yes?

meloƞ

Yes?

I Saw u typing :)

03:40

I Just woke Up

meloƞ

Its 5:40 AM

03:40

Help

Yeeeh me 2, sometimes i awake at midnight hahaha

meloƞ

Solely talking about dynamic libs (sqlite, steamapi, avformat, sdl2) - i'll Check If i can modify my cmakelists to do that for me. Thanks ❤️

Maybe with a Post_build Copy ? Hmm Coffee and 10 hours Work before in can Check monkaStop

meloƞ

Solely talking about dynamic libs (sqlite, steamapi, avformat, sdl2) - i'll Check If i can modify my cmakelists to do that for me. Thanks ❤️

same deal

04:02

environment should handle it and an install/deploy step should be responsible for copying it into the build dir

04:03

if you're using MSYS2 you should be able to start it through you env's terminal because it has /*/lib and whatnot added to the library search paths

04:03

and then use a separate deploy step

04:03

those DLLs are not built alongside ddnet

ddnet-maps

[ddnet-maps:master] 1 new commit

f8a9ebd A goup - ddnet-maps

Ewan

and then use a separate deploy step

I'll Just add a Script to move the necessary Files to my build folder, Just annoying i have to Launch a Script every time i Clean rebuild. I wanna get rid of IDE's, they take to much work off of me and i feel like i know nothing about the actual process when you have a program doing it for you

im not single anymore justatest

https://www.phoronix.com/news/NVK-Implicit-Pipeline-Caching

NVK Vulkan Driver Adds Implicit Pipeline Caching To Boost DXVK Perf...

Adding to the impressive number of features to be found in this quarter's Mesa 24.1 release is now the open-source NVIDIA 'NVK' Vulkan driver supporting implicit pipeline caching.

@Learath2 btw my coworker, aided by our findings found the issue

08:48

the issue was that some structs were misaligned, we use the Layout from rust to know the sizes and aligns of the data we create, but made a wrong assumption, we needed to also pad to align at the end of structs

08:49

tldr, we thought 28 was the correct size for example but llvm assumed 32

Ryozuki

the issue was that some structs were misaligned, we use the Layout from rust to know the sizes and aligns of the data we create, but made a wrong assumption, we needed to also pad to align at the end of structs

So was it that you were allocating the wrong stack size as I hypothesized?

yep

08:52

u hit the nail

Ryozuki

im not single anymore justatest

LETSGOOOOOOOO

Ryozuki

im not single anymore justatest

WAIT WHAT? that's why our epyc guy i never active on discord lately

09:28

xD

Dont Show your discord Chat History or she s gonna want to learn Rust and never leave her room again

<ChillerDragon> @Krimp do you still play ctf? I made a new release of ddnet 0.7 that is bug free as far as I know so I would appreciate any bug reports :)

09:34

<ChillerDragon> https://chillerdragon.github.io/ddnet7-client/

meloƞ

Dont Show your discord Chat History or she s gonna want to learn Rust and never leave her room again

plot twist. in fact it's Voxel acting like a girl to get ryo away from rust. His secret master plan all along

1

Jupstar ✪

plot twist. in fact it's Voxel acting like a girl to get ryo away from rust. His secret master plan all along

He would've started with me fr

no its an ai girlfriend

<ChillerDragon> @fokkonaut i dmed u btw

chiller

10:07

wtf happened in ur 2nd screenshot xd

<ChillerDragon> wot

looks like photoshop

<ChillerDragon> a

ws-client

<ChillerDragon> @Krimp do you still play ctf? I made a new release of ddnet 0.7 that is bug free as far as I know so I would appreciate any bug reports :)

Yes I still play vanilla thanks to your client, and this version seems to fix the issues that I had, the only minor oddities I notice are that 1.) On vanilla 0.7 servers (dm and ctf at least) no sound is made when I hook another tee now (the sound plays on 0.6 servers in the client) and 2.) The value for the alpha channel on the skin markings goes to 0 every time I reset client on any skin that doesn't have a json file setup (idk if that's true in vanilla 0.7 or not)

<ChillerDragon> its gimp

10:08

<ChillerDragon> i wanted to show the green checkmark and console :D

10:09

<ChillerDragon> @Krimp you have those bugs with the new version from today?

10:09

<ChillerDragon> those should be fixed now

I may have accidentally downloaded the wrong 18.1 in that case, I'll try again

<ChillerDragon> 18.1 should not have those bugs :(

10:15

<ChillerDragon> ah shit i can reproduce nice find thanks @Krimp

Yeah I am having those issues still in 18.1

10:15

Oh okay nice np ty for client!

Ryozuki

im not single anymore justatest

I saw your commit history crumbling. I knew something was up!!

@ChillerDragon i cant get it running on my linux system.

./DDNet: error while loading shared libraries: libavformat.so.58: cannot open shared object file: No such file or directory

any fix for this? (edited)

self compile

I don't have the source code unfortunately. I compiled the normal ddnet yesterday and everything works perfectly

You are missing a library, the specific version chiller linked to. You'll need to get that

11:04

It's ffmpeg btw

I downloaded that from github actions. Was hoping that is portable.

11:12

Maybe I have to do the Linux one manually then. What’s the way to go here? Link statically? What cmake flag is that? (edited)

u could also disable video rendering

The source code is at ChillerDragon/ddnet on GitHub in the pr_07_client branch

Replying to @Gotie I don't have the source code unfortunately. I compiled the normal ddnet …

1

https://github.com/ChillerDragon/ddnet/tree/pr_07_client

GitHub - ChillerDragon/ddnet at pr_07_client

Chillers contribution fork of DDraceNetwork, a cooperative racing mod of Teeworlds - GitHub - ChillerDragon/ddnet at pr_07_client

I am curious on how to publish a release in general

Replying to @Jupstar ✪ u could also disable video rendering

chillerdragon

I am curious on how to publish a release in general

ddnet-scripts contains all stuff related to publishing

Yes I know -.-

but if u want stuff like auto updater.. that is more annoying

Idk if people even use the Linux binary release

11:16

Is it even good style to statically link libraries in a binary release?

11:16

I guess the intended way is everything is managed by the package manager

chillerdragon

Is it even good style to statically link libraries in a binary release?

why not xd

ws-client

<ChillerDragon> That CI fail is suspicious. https://github.com/ddnet/ddnet/actions/runs/8825297278/job/24229302734?pr=8236

<ChillerDragon> the pr added an unused method that is never called. And the CI failed in the admin integration test. Are the integration tests getting unstable?

See #8261, the same test also failed with just a translation change. I assume it's related to the recently changed integration test script/map in #8231.

https://github.com/ddnet/ddnet/issues/8261 https://github.com/ddnet/ddnet/pull/8231

Integration test failure with `admin message not found in server lo...

[] Setup [] Launch server [] Launch client 1 [] Start demo recording [] Launch client 2 [] Test chat and chat commands [] Test rcon commands [] Stop demo recording [] Test map change [] P...

Add check for correct finish time in integration_test.sh by Zwelf ·...

Checklist Tested the change ingame Provided screenshots if it is a visual change Tested in combination with possibly related configuration options Written a unit test (especially base/) or add...

there are static link fans and dyn link fans

11:21

@ChillerDragon as far as i can see the ddnet release from website static links ffmpeg too

11:21

Dynamic Section:
  NEEDED               libfreetype.so.6
  NEEDED               libSDL2-2.0.so.0
  NEEDED               libdl.so.2
  NEEDED               libvulkan.so.1
  NEEDED               libGL.so.1
  NEEDED               libnotify.so.4
  NEEDED               libgdk_pixbuf-2.0.so.0
  NEEDED               libgio-2.0.so.0
  NEEDED               libgobject-2.0.so.0
  NEEDED               libglib-2.0.so.0
  NEEDED               libcurl.so.4
  NEEDED               librt.so.1
  NEEDED               libm.so.6
  NEEDED               libpthread.so.0
  NEEDED               libc.so.6
  NEEDED               ld-linux-x86-64.so.2
  RUNPATH              $ORIGIN:/usr/local/lib:/ddnet-source/ddnet-libs/curl/linux/lib64:

11:21

that's all dynamic deps

Rust biased axaxaxax

Replying to @Jupstar ✪ why not xd

11:30

Do you know how to link ffmpeg statically? Or do I have to look it up ._.

i dont know by heart

11:33

maybe we have a cmake option for it

<ChillerDragon> I have a bunch of stale discussions in issues and prs and I am slowly getting impatient :/

11:53

<ChillerDragon> I am waiting for a yes or no + why in here since last year https://github.com/ddnet/ddnet/issues/7480

Extend redirect to include ip not only port · Issue #7480 · ddnet/d...

Currently the server can tell a client to reconnect to a different port. I would like to also allow redirection to another ip. heinrich was concerned about security. I would like to reopen this dis...

11:54

<ChillerDragon> This pr is ready since 2 weeks https://github.com/ddnet/ddnet/pull/8151

Enforce UpperCamelCase for local variables by ChillerDragon · Pull ...

For now it is only checking local variables. There are also member/private/protected/parameter/global and so on variables. There are a lot of exceptions to show 0 warnings on the current code base ...

11:55

<ChillerDragon> this is somewhat blocking a lot of my projects and nobody responded to my bump in february https://github.com/ddnet/ddnet/issues/7777

Source-level moddability discussion · Issue #7777 · ddnet/ddnet

How much do we want to support moddability in the DDNet codebase? Note that this doesn't talk about the network protocol but about the actual code.

tja

I don't reply to them because you won't like my answer and you already spent so much time on all of them

14:01

Actually I can reply to one

i'd love to have 7480 - espacially since blockworlds™ wants to expand ( and having a server redirect on multeasymap might help for once kek

)

For the uppercamelcase thing, I absolutely hate that we are adding another long running task to our already long asf CI run

Learath2

For the uppercamelcase thing, I absolutely hate that we are adding another long running task to our already long asf CI run

if its all about readability, refactor everything CAndEveryClassWouldLookLikeThis xD

Ryozuki

im not single anymore justatest

oh god you have bf now

meloƞ

i'd love to have 7480 - espacially since blockworlds™ wants to expand ( and having a server redirect on multeasymap might help for once kek

)

redirecting to arbitrary servers is not nice tho imagine u type google.com and land on pornhub

Jupstar ✪

redirecting to arbitrary servers is not nice tho imagine u type google.com and land on pornhub

i wouldnt complain. justatest

Jupstar ✪

redirecting to arbitrary servers is not nice tho imagine u type google.com and land on pornhub

You could, nothing in the HTTP protocol stops that

Jupstar ✪

redirecting to arbitrary servers is not nice tho imagine u type google.com and land on pornhub

i type twitter and dat shit redirects to some xxx

meloƞ

i wouldnt complain. justatest

ur parents behind u

.,.

Learath2

You could, nothing in the HTTP protocol stops that

yes, but u'd probably not want it maybe u typed goo.gl and it redirected you to google... all fine, that's what u wanted (edited)

14:17

but random servers is rarely the intended use case

14:17

also i guess it's javascript doing it, isnt it?

Jupstar ✪

also i guess it's javascript doing it, isnt it?

An HTTP 301 can take you anywhere