[bug #39306] Contrib: grub-list-kernels.pl script

[Vladimir Serbinenko]

Update of bug #39306 (project grub):

                  Status:                Wont Fix => Invalid                

    _______________________________________________________

Follow-up Comment #8:

This is not a right place to contribute. This is a bug tracker. For
contributions use address@hidden The script is not acceptable as-is as it
would encourage bad practice (use of indices). Another problem is the
language: we don't use perl at all and introducing a new dependency for a
secondary script is not an option. Changing to 'Invalid' since it's not a bug
report. The task you've chosen is not fully solvable since menus can be
generated dynamically, consider following:
for x in (*); do
   if [ -f $x/vmlinuz ]; then
      menuentry "Boot vmlinuz from $x" --menuentry-id "$x" "$x" {
            set root=$1
            linux /vmlinuz
      }
    fi
done
You can't even know how mayn entries this generates and depending on this all
the following indices may shift as well, yet another reason not to use
indices.
I'd happily discuss this and other issues but as I said this is not a right
place.
As to whether you want to walk away, it's really your choice, I was just
pointing out that your solution doesn't fit into correct usage of concerned
parts.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?39306>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/