thias / glim

GRUB Live ISO Multiboot
638 stars 141 forks source link

Limit the scope of README parsing in glim.sh #109

Closed slowpeek closed 1 year ago

slowpeek commented 1 year ago

glim.sh does parse the list to precreate the distro subdirs. With the markers we limit the scope of parsing to the bare minimum.

References: https://stackoverflow.com/questions/4823468/comments-in-markdown


The $args thing there is to make lines shorter. Loooong command inside $(..) looks not healthy

thias commented 1 year ago

Thanks for making it more robust! But I already had in mind some further changes, which will require changing the parsing further: I wanted to change the code block to become a list along these lines:

* [almalinux](https://almalinux.org/) (only Live Media)
* [debian](https://www.debian.org/CD/live/) (only live install or `mini.iso`)
* [gparted](https://gparted.org/)

I find it still quite readable as plain text, and providing links has been a recurring request from users, though to be kept quite generic in order to avoid breaking for minor distro website reorganizing. The example above has GParted (trivial download), AlmaLinux (quick find) and Debian (really not obvious and hasn't changed in decades, so link with path). It's also the perfect place for very minor details, such as the supported spins/variants/flavors.

Any thoughts about also going in that direction?

slowpeek commented 1 year ago

Any thoughts about also going in that direction?

Links are great and with such plain format it is easy to parse.