Skip to content

Avoid udpserver.c appears in a random place in the doc - #23005

Open
couet wants to merge 1 commit into
root-project:masterfrom
couet:legacy-net
Open

couet wants to merge 1 commit into
root-project:masterfrom
couet:legacy-net

Conversation

@couet

@couet couet commented Aug 4, 2026

Copy link
Copy Markdown
Member

Avoid udpserver.c appears in a random place in the doc: https://root.cern.ch/doc/master/udpserver_8c.html

@couet
couet requested a review from bellenot August 4, 2026 10:09
@couet couet self-assigned this Aug 4, 2026
@couet
couet requested a review from dpiparo as a code owner August 4, 2026 10:09
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 19h 25m 38s ⏱️
 3 853 tests  3 851 ✅ 0 💤 2 ❌
79 408 runs  79 397 ✅ 9 💤 2 ❌

For more details on these failures, see this check.

Results for commit 5e0abd8.

@dpiparo

dpiparo commented Aug 4, 2026 •

Copy link
Copy Markdown
Member

Just wondering here: do we need the udp server at all? @jblomer
Is it also perhaps an opportunity to rediscuss the TUDPSocketclass and usages?

@couet
couet requested a review from jblomer August 18, 2026 08:18
@jblomer

jblomer commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

The tutorial is already in the legacy space, so it's a small step to remove it. In general, I think we should rather not advertise ROOT as a tool for network programming. So I would have a preference for removing the tutorial.

We can also deprecate TUDPServer for ROOT7, I think. We seem to not using it ourselves.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants