DDraceNetwork - developer

DDraceNetwork

Development / developer

Development discussion. Logged to https://ddnet.org/irclogs/ Connected with DDNet's IRC channel, Matrix room and GitHub repositories — IRC: #ddnet on Quakenet | Matrix: #ddnet-developer:matrix.org GitHub: https://github.com/ddnet

Between 2023-09-30 00:00:00Z and 2023-10-01 00:00:00Z

This is the most confusing thing I've ever seen

Image attachment

github-merge-queue[bot]

[ddnet:master] 2 new commits

fcffac6 Fix the fix to dead reckoning - Learath2 1beab04 Merge pull request #7267 from Learath2/pr_fixfixreckon - heinrich5991

00:15

[ddnet/ddnet] Pull request closed: #7267 Fix the fix to dead reckoning

Learath2

This is the most confusing thing I've ever seen

The number on top of your own tee is not your client id but it's your strong weak id, it made it look like both tees had the same client id, made me go on a wild goose chase for 20 minutes

damn

Ryozuki

first

Image attachment

06:15

Embedded image

i remember this

@Ryozuki do u have rustup installed?

OF COURSE BRO what kind of a question is that

07:05

owo

07:06

New emoji to add to ur irc client btw ChillerDragon

Ewan

OF COURSE BRO what kind of a question is that

just asking

Image attachment

is noby fng gone from the server browser for anyone else? its been like that for days now

owo

🩳

I see it

MilkeeyCat

@Ryozuki do u have rustup installed?

yes man i told u to not use system rust and use rustup and install RA with it

08:33

u dont listen

08:33

justatest

08:33

owo

Ryozuki

yes man i told u to not use system rust and use rustup and install RA with it

The hell is RA

Ryozuki

u dont listen

My bad

feelsbadman

08:57

Now time to read how to install rustup package poggers2

poggers2

ra is rust analyzer

Oh

08:58

Oke

install rustup from web

08:58

the curl command it shows

Did u not emerge it?

I installed it from the repositories

09:18

I'd probably also emerge it on gentoo

https://smallcultfollowing.com/babysteps/blog/2023/09/29/polonius-part-2/

MilkeeyCat

Did u not emerge it?

rustup? nop

09:23

i like my rustup from web

09:23

linux rust binaries are optimized with lto + bolt btw

09:23

its probs faster than ur local build

09:23

and pgo*

Where did u get rust?

i just told u

09:24

rustup

I wouldn't emerge rustup on gentoo

https://rustup.rs/

rustup.rs - The Rust toolchain installer

The Rust toolchain installer

09:24

curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh

09:24

this on ur local user

09:24

its a per user install

09:24

rustup is awesome cuz u can have lot of rust versions

09:25

and for other toolchains

09:25

if u havent do

09:25

rustup component add rust-src

Btw

maybe u were missing t hat for RA

09:25

it needs it

Learath2

I wouldn't emerge rustup on gentoo

what's the point of gentoo then?

https://rust-analyzer.github.io/manual.html#installation

rust-analyzer: building a better Rust IDE

firefox is also pgo-ed

Yesterday i spent 3 hours installing rust by emerging :)

and chromium as well

heinrich5991

what's the point of gentoo then?

highly customizable linux distro, kernel tuning, u can customize all packages via use flags and custom patches

heinrich5991

what's the point of gentoo then?

You can teeeechnically have system rust from rustup, but there are some annoying issues with it

also ebuild is awesome

Learath2

You can teeeechnically have system rust from rustup, but there are some annoying issues with it

you can install rustup from emerge without also using system rust

more than pkg thing from arch imho

09:27

and i made packages on both

09:27

(from my xp maintaing ddnet)

09:27

maintaining

heinrich5991

you can install rustup from emerge without also using system rust

at least on arch

09:28

still think I'm missing something

Wym without also using system rust?

if you install rustup

09:28

on arch

09:28

you still use rustup to install rustc, cargo etc.

not on gentoo

it's just rustup that's installed using the package manager

09:28

ah, weird

iirc

09:28

@MilkeeyCat what use flags u added to the rust pkg

09:29

dev-lang/rust clippy rust-analyzer rust-src rustfmt

09:29

?

09:29

btw if u add system-llvm u may not need to compile 2 versions of llvm

09:29

anyway make sure u added rust-src

09:29

RA needs it

In a generic installation of Rust (one that is not done with Gentoo's Rust package) things are usually updated and installed with dev-util/rustup.

Thats what am gonna do

heinrich5991

you still use rustup to install rustc, cargo etc.

Since gentoo compiles everything, it needs a system rust

aah

09:30

I see, that's the problem

yeah gentoo packages that use rust use system rust

Ryozuki

@MilkeeyCat what use flags u added to the rust pkg

I just ran emerge dev-lang/rust :clueless:

@MilkeeyCat do u know how to add per package use flags?

it's a bootstrapping problem

You can have rustup, it's just that ebuilds will be tested against the rust package, so you can get weird issues

@MilkeeyCat did u ever edit portage package.use

09:33

https://wiki.gentoo.org/wiki//etc/portage/package.use

09:33

https://wiki.gentoo.org/wiki/Handbook:Parts/Working/USE#Declaring_USE_flags_for_individual_packages

/etc/portage/package.use is a folder, not a file

09:34

and it has cpu-flags file :clueless:

yeah

09:34

all files inside

09:34

are like package.use

09:34

u can organize better then

09:34

make another file

09:35

i would call it 00-general

so i can name it however i want

or smth

09:35

yeah

09:35

anything

09:35

00 is to change the order

09:35

in case u add a package multiple times in multiple files iirc the last one is used

09:35

or maybe they are merged

soooooo, i have to delete emerged rust and install everything from web

09:36

ye?

owo

no!

09:36

keep ur system rust

pepeW

install rustup

09:36

system rust will be used for gentoo packages that use rust

09:36

but if u dont have any system pkgs that depend on rust well u can remove it if u want

09:36

(its fine to have both system rust and rustup)

i better dont delete it coz i dont want to reinstall gentoo again

?? zs

09:37

xd

09:37

it wont let u delete it if other pkgs depend on it

09:38

simply --deselect it

09:38

and if nobody depends on it

09:38

--depclean will remove it

09:38

u can also check if something depends on it with equery

09:38

equery d rust

09:38

iirc

thanks god theres protection from such idiots like me owo

owo

i have a spare ryzen 3200g

09:39

i think it can be a good initial fit for a home server

09:39

i need a motherboard

tee_thinking

Image attachment

well

09:39

u can still --deselect it

09:40

portage will keep it as long as a package depends on it

09:40

u only keep packages in world file that u use directly

so how do i have to install rustup? xD

Ryozuki

curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh

go to ur non root usual user

09:41

and run that

oke

09:44

justatest

Image attachment

[ddnet/ddnet] Pull request closed: #7241 Fix mangled highlighted text in server browser

09:49

github-merge-queue[bot]

[ddnet:master] 2 new commits

95675a9 Fix mangled highlighted text in server browser - Robyt3 aacfc9a Merge pull request #7241 from Robyt3/UI-LabelStreamed-HighlightedFix - def-

MilkeeyCat

justatest

continue

09:50

if ur non root its fine

09:50

well as root would be fine too

09:50

rustup doesnt install rust in usr bin

09:51

it installs everything in /home/user/.cargo/bin/cargo

09:51

for example

09:51

so u add the bin dir to path in bashrc

09:51

and ur done

github-merge-queue[bot]

[ddnet:master] 6 new commits

8f8c9a2 Add a popup for when video render is done. - furo321 79c503e Fix requested changes - furo321 74567a1 ability to show local keypresses only - dobrykafe 7c710dd add another value instead of new variable - dobrykafe 3e2893e Merge pull request #7182 from dobrykafe/pr-show-direction - def- 1452b36 Merge pull request #7172 from furo321/render-done-popup - def-

09:58

[ddnet/ddnet] Pull request closed: #7182 Ability to show local keypresses only

09:58

[ddnet/ddnet] Pull request closed: #7172 Add a popup for when video render is done

it works

Image attachment

10:02

but not in tmux yet pepeW

pepeW

tmux sux, use zellij

dont u dare to say anything bad about my boi tmux

MilkeeyCat

but not in tmux yet pepeW

pepeW

wdym

10:18

maybe restart ur tmux session

10:18

btw soon is hacktoberfest

10:19

https://hacktoberfest.com/

Hacktoberfest 2023

Hacktoberfest: a month-long celebration of open-source projects, their maintainers, and the entire community of contributors.

Ryozuki

wdym

when i start tmux it changes $PATH pepeW

pepeW

(edited)

Image attachment

i think tmux has its own thing?

Ryozuki

maybe restart ur tmux session

i restarted pc xd

check ur .bashrc

10:21

for a line that sets ur path

10:21

and fix tmux

10:22

@MilkeeyCat ah

10:22

add the line to .profile

Image attachment

whats this?

10:23

ah

10:23

did u add that

it was added after rustup installation

if [ -f /etc/profile ]; then
    PATH=""
    source /etc/profile
fi

10:25

try adding this

10:25

to .bash_profile

10:25

idk about other shells i use bash

10:26

https://superuser.com/questions/544989/does-tmux-sort-the-path-variable

Does tmux "sort" the PATH variable?

I have a reproducible problem: set up my PATH in Bash .profile start tmux by tmux, tmux attach or any variant echo $PATH and see it with the same components but in different order How to stop this?

10:26

but well u should research this urself

10:26

this is not a rust problem

Ryozuki

if [ -f /etc/profile ]; then
    PATH=""
    source /etc/profile
fi

i jsut had to put same line from .profile in .zprofile xD

10:49

and everything works

When does deen make a studio like hypixel & makes his own tw-like game

github-merge-queue[bot]

[ddnet:master] 2 new commits

e4ddbaf add export image button - Marmare314 359d9ab Merge pull request #7176 from Marmare314/export-image - def-

11:25

[ddnet/ddnet] Pull request closed: #7176 add export image button

[ddnet/ddnet] Pull request opened: #7268 Various refactoring of the engine server browser, remove game includes from engine, minor performance improvement

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-addres...

[ddnet/ddnet] Pull request closed: #7268 Various refactoring of the engine server browser, remove game includes from engine, minor performance improvement

12:17

github-merge-queue[bot]

[ddnet:master] 9 new commits

e78305e Remove unused m_aFilter(Gametype)String variables - Robyt3 6753e37 Move function definitions to remove and organize includes - Robyt3 721a55c Remove game menus includes in engine - Robyt3 dcff6d9 Add name for enum EClientScoreKind - Robyt3 034962a Add enum ERankState to replace magic numbers - Robyt3 28d4451 Add local variable Info to reduce duplicate code - Robyt3 7e8442d Move and rename UpdateFilteredPlayers function, mark as const - Robyt3 cf35594 Extract UpdateServerFriends function, cache friends on server - Robyt3 18d2f1a Merge pull request #7268 from Robyt3/Serverbrowser-Refactoring-Various - def-

MilkeeyCat

i jsut had to put same line from .profile in .zprofile xD

Have you tried fish yet? I like it better than zsh

13:11

Well I use bash still but if I wanted to use a more “advanced” shell fish is the one I liked more

[ddnet/ddnet] Pull request opened: #7269 Fix large FPS drop rendering server details of favorite server

The server list was being resorted every frame when the server details of a favorite server were being rendered, which was causing the FPS to drop by a factor of 5.

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 indexin...

13:59

[ddnet/ddnet] Pull request closed: #7087 Add streamer mode to avoid leaks

13:59

github-merge-queue[bot]

[ddnet:master] 2 new commits

ca8d9d4 Add streamer mode to avoid leaks - ChillerDragon c731d53 Merge pull request #7087 from ChillerDragon/pr_streamer_mode - def-

Learath2

Have you tried fish yet? I like it better than zsh

I didn't :o what did u like more in fish than in zsh?

MilkeeyCat

I didn't :o what did u like more in fish than in zsh?

The syntax feels more human and the entire thing more polished

14:45

The completion/syntax highlighting as you type is pretty nice

14:47

I still use bash myself though because I didn't feel like installing any shell on all the machines I use

fellow fish user?

14:50

oh u use bash

14:51

fish not being a login shell sorta blows

this

1

ive been using gnucash and so far its nice

15:03

not like i have xp using any other

15:03

i wish my bank had a export option in a format thats not excel

15:03

imaging having standards

Learath2

I still use bash myself though because I didn't feel like installing any shell on all the machines I use

have your heard about ansible? gigachad

gigachad

MilkeeyCat

have your heard about ansible? gigachad

gigachad

Sure

[ddnet/ddnet] Pull request opened: #7270 Add export button to sounds

As suggested in #7176.

Export button is now available for sounds.
Filename is now by default the assets name
Use dbg_assert for Invalid image format

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

...

i finally have right time

15:41

ok, maybe i didnt pepeW

pepeW

(its 6:40pm) (edited)

Image attachment

@Robyt3 I think you might have broken something with the recent server browser changes. It seems like various server are missing on latest git master. Here's a comparison on the DDNet tab.

Image attachment

Image attachment

furo

@Robyt3 I think you might have broken something with the recent server browser changes. It seems like various server are missing on latest git master. Here's a comparison on the DDNet tab.

I'll take a look later, thanks

github-merge-queue[bot]

[ddnet:master] 2 new commits

f07e7fe Add export button to sounds - furo321 d55822b Merge pull request #7270 from furo321/editor-export-sound - Robyt3

16:30

[ddnet/ddnet] Pull request closed: #7270 Add export button to sounds

MilkeeyCat

ok, maybe i didnt pepeW

pepeW

(its 6:40pm) (edited)

ok now fr fr

furo

@Robyt3 I think you might have broken something with the recent server browser changes. It seems like various server are missing on latest git master. Here's a comparison on the DDNet tab.

Actually wasn't my changes that caused this. Bisect showed https://github.com/ddnet/ddnet/commit/c4d325e09baeff27ab800283dca4fe8cf6aff79a is the first bad commit. The servers which are not shown have clients which don't have the "afk" attribute, so their info is considered invalid.

Show if your friend is AFK through friends list · ddnet/ddnet@c4d325e

resolve mege conflict AFK -> Afk (for consistency)

17:40

Would make more sense in my opinion to add backwards compatibility for missing attributes, but it looks like everything else is checked very strictly

Oh, sorry. Yeah, the attribute should definitely be optional and be set to false if missing.

17:52

I'll create a PR to fix it.

[ddnet/ddnet] Pull request opened: #7271 Make IsAfk attribute optional

Otherwise will servers that don't send it be considered invalid and not shown in the browser.

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...

github-merge-queue[bot]

[ddnet:master] 2 new commits

0025b6c Fix large FPS drop rendering server details of favorite server - Robyt3 49a5a59 Merge pull request #7269 from Robyt3/Serverbrowser-Favorite-Resort-Fix - def-

18:36

[ddnet/ddnet] Pull request closed: #7269 Fix large FPS drop rendering server details of favorite server

Ewan

fish not being a login shell sorta blows

wdym, not a login shell? I use it as my login shell

Ryozuki

i wish my bank had a export option in a format thats not excel

python can read excel stuff

18:43

(if you need a way to read it)

Learath2

I still use bash myself though because I didn't feel like installing any shell on all the machines I use

on ddnet servers it's already installed

and on all my servers, too

github-merge-queue[bot]

[ddnet:master] 2 new commits

3b2389d Make IsAfk attribute optional - furo321 c8401aa Merge pull request #7271 from furo321/afk-optional - def-

18:44

[ddnet/ddnet] Pull request closed: #7271 Make IsAfk attribute optional

for l in (cat testmsgs); echo $l | perl -e 'my @arr = split " ", <STDIN>; foreach ( @arr ) { printf("%0.2x ", $_); }; printf("\n");'; end | sed 's/^/\\\\x/' | sed 's/ /\\\\x/g' | sed 's/\\\\x$//'

cursed teehistorian debugging tool

??!?

[ddnet/ddnet] Pull request opened: #7272 Don't run GET_INT on afk attribute in ProcessServerInfo

This line shouldn't be here. It causes really weird issues on Non-DDNet servers. !image

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 added coverage to integration test
[ ] Considered possible null pointers and out of bounds a...

heinrich5991

??!?

It’s my awful attempt at getting a binary message out of a teehistorian file without compiling stuff

19:35

(abusing teehistorian_dump along the way

heinrich5991

wdym, not a login shell? I use it as my login shell

you must be very lucky, it does a bunch of POSIX incompatible stuff like exiting on failure (which will cause a panic if it’s the first process)

20:00

dunno where exactly these incompatibilities are documented but both arch and gentoo wikis mention how it’s not a POSIX compatible shell (gentoo’s the only one that says it will outright break your system if you use it as a login shell)

20:01

i think it’s intentionally nonstandard so it can address flaws of normal shells. just wish they had a mode or something where it can be used as the login shell

it can absolutely be used as a login shell

it might even warn if it detects it’s being used as one?

maybe not on gentoo

20:01

but definitely on arch

20:01

it does not (on arch)

arch is where i initially discovered it couldn’t

20:02

it works as the initial process?

after I enter my username and password I get to fish

which init system do u use

systemd

oh yeah

20:03

weird

20:03

how did you set it up? chsh broke iirc

chsh -s /usr/bin/fish user

justatest

also on all my servers btw

u have magic fish

I think you have a fear of fish ^^

heinrich5991

also on all my servers btw

maybe i have it setup as the login shell on my vps actually… I could be thinking of something else

arch doesn't even warn against it anymore

oh yeah maybe they just fixed it

Ewan

i think it’s intentionally nonstandard so it can address flaws of normal shells. just wish they had a mode or something where it can be used as the login shell

did this or smth

they did not

20:05

it still works in a good way

20:07

oh, fish is partially written in rust these days

(edited)

rust btw

20:08

blazingly fast

20:08

thread safe

20:08

memory efficient

20:08

and also more BLAZINGLY FAST (edited)

fish intro says this

20:13

Warning Setting fish as your login shell may cause issues, such as an incorrect PATH. Some operating systems, including a number of Linux distributions, require the login shell to be Bourne-compatible and to read configuration from /etc/profile. fish may not be > suitable as a login shell on these systems.

(edited)

how is rust debugging?

you don't "debug" rust, you just compile it and it runs perfectly

github-merge-queue[bot]

[ddnet:master] 2 new commits

a3064ae Don't run GET_INT on afk attribute in ProcessServerInfo - furo321 e0d906d Merge pull request #7272 from furo321/afk-optional2 - def-

21:05

[ddnet/ddnet] Pull request closed: #7272 Don't run GET_INT on afk attribute in ProcessServerInfo

アリヂン

you don't "debug" rust, you just compile it and it runs perfectly

what

what

1

Ewan

how is rust debugging?

eeh, anything beyond C is a certain degree of annoying to debug

21:41

I've had the pleasure of debugging some async rust and the 76 deep backtrace was extremely annoying

[ddnet/ddnet] Pull request opened: #7273 revert quad selection correctly fixes #7248

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-ad...

Learath2

I've had the pleasure of debugging some async rust and the 76 deep backtrace was extremely annoying

oh sounds great

21:45

most c/cpp debugging tools are probably completely useless on it right

Ewan

most c/cpp debugging tools are probably completely useless on it right

there is a gdb plugin that makes it bearable

oh good

@heinrich5991 classic bash is the best for me, but heard alot about zsh, only thing is it looks slow af

22:23

with all the shitty stuff it comes with

22:23

(with also oh-my-zsh) (edited)

yeah I don’t love zsh

ReiTW

@heinrich5991 classic bash is the best for me, but heard alot about zsh, only thing is it looks slow af

fish is almost bash with good defaults

try it some time

Ewan

oh good

use the rust-lldb took

23:10

tool

1

[ddnet/ddnet] Pull request opened: #7274 Use m_aInputData for dummy in cl_show_direction

Now you would be able to see which keys your dummy is holding while in freeze. !image

Checklist

[x] Tested the change ingame
[x] 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...

Exported 290 message(s)