Skip to content

Update link to upstream iwd documentation page#820

Merged
classabbyamp merged 1 commit into
void-linux:masterfrom
grtcdr:patch-1
Nov 17, 2024
Merged

Update link to upstream iwd documentation page#820
classabbyamp merged 1 commit into
void-linux:masterfrom
grtcdr:patch-1

Conversation

@grtcdr

@grtcdr grtcdr commented Nov 17, 2024

Copy link
Copy Markdown
Contributor

The old link redirects to the kernel's rather funny 404 page.

@grtcdr

grtcdr commented Nov 17, 2024

Copy link
Copy Markdown
Contributor Author

403 link in src/config/graphical-session/wayland.md line 61 is causing the check links step to fail. Perhaps it should be changed to https://wiki.gnome.org/Initiatives(2f)Wayland(2f)Applications.html.

Official GTK documentation also links to this (now) forbidden resource: https://docs.gtk.org/gtk3/wayland.html

@grtcdr

grtcdr commented Nov 17, 2024

Copy link
Copy Markdown
Contributor Author

Running vmdfmt -w iwd.md (or check.sh) does not affect the file, so formatting shouldn't be failing in CI.

@classabbyamp

classabbyamp commented Nov 17, 2024

Copy link
Copy Markdown
Member

I get the same diff as CI from vmdfmt, not sure what you're doing locally

The old link redirects to the kernel's rather funny 404 page and the new
link says "For most up-to-date information on network settings see the
iwd.network(5) man page shipped with IWD (source), which incorporates most
of this page's content now."
@classabbyamp

Copy link
Copy Markdown
Member

the new page says

For most up-to-date information on network settings see the iwd.network(5) man page shipped with IWD (source), which incorporates most of this page's content now.

so we should just link iwd.network(5) instead

@classabbyamp classabbyamp merged commit 1008fd6 into void-linux:master Nov 17, 2024
@grtcdr

grtcdr commented Nov 17, 2024

Copy link
Copy Markdown
Contributor Author

I get the same diff as CI from vmdfmt, not sure what you're doing locally

I wouldn't know either. I cloned the freshly-created fork, ran vmdfmt as stated in CONTRIBUTING.md and it never found an issue with the formatting.

so we should just link iwd.network(5) instead

Even better :)

@classabbyamp

Copy link
Copy Markdown
Member

were you on the right branch?

@grtcdr

grtcdr commented Nov 17, 2024

Copy link
Copy Markdown
Contributor Author

Oh my, I wasn't. Sorry about that. I really need to add git branch information to my PROMPT (and avoid the web editor because it's plain confusing)

@grtcdr grtcdr deleted the patch-1 branch November 17, 2024 14:49
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.

2 participants