Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken links #4

Open
layderv opened this issue Dec 21, 2018 · 5 comments
Open

Broken links #4

layderv opened this issue Dec 21, 2018 · 5 comments

Comments

@layderv
Copy link

layderv commented Dec 21, 2018

Some links in your README.md are broken, for example the one to demesg (https://github.com/cirosantilli/linux-cheat/blob/master/demesg.md)

@cirosantilli
Copy link
Owner

Fixed the dmesg, but lazy to go over all now.

I wish we had an automated way to check it.

@layderv
Copy link
Author

layderv commented Dec 24, 2018

With your README.md in the current folder, I run:

files=$(cat README.md | perl -n -e '/\[.+\]\((.+)\)/ && print $1 . "\n"')
for f in $files; do
    u=$f
    if [[ ! $f =~ ^(https://) ]]; then u="https://github.com/cirosantilli/linux-cheat/tree/master/$f"; fi
    wget --spider -q "$u" || echo "$f does not exist"
done

and I see that:

android.md does not exist
runs-linux.md does not exist
FUSE.md does not exist
/dev/loop.md does not exist
/dev/sr0.md does not exist
make/ does not exist
env.md does not exist
chroot.sh does not exist
book.md does not exist
gnuplot/ does not exist
xmctrl.md does not exist
libnotify.md does not exist
zenity.md does not exist
iconv.sh does not exist
vagrang/ does not exist
graphiz.md does not exist
anacron does not exist

You might want to verify them :)

cirosantilli added a commit that referenced this issue Dec 26, 2018
@cirosantilli
Copy link
Owner

Thanks, I've now fixed those at: 360e8c4

This repo is messy by design I guess.

@layderv
Copy link
Author

layderv commented Dec 26, 2018

I run the same script and:

chroot.sh does not exist
book.md does not exist
gnuplot/ does not exist

and the README should then be fixed :)

@cirosantilli
Copy link
Owner

Update again! :-)

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

No branches or pull requests

2 participants