Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

Tawal · Le 08/03/2023, à 12:16

Hello,

Je regardais un peu le paquet isenkram.
Dans le man isenkram-lookup, je trouve cette commande (pour trouver les modalias de la machine ) :

cat $(find /sys -name modalias) | sort -u

Je ne la trouve pas top et elle retourne des erreurs de permissions sur certains "répertoires".
Puis dans une autre documentation (tout en bas de cette page), je trouve celle-ci :

cat $(find /sys/devices/|grep modalias)

Elles est plus propre en retour, mais bon un pipe, une substitution de commande ...
Le pipe est évitable avec le test -name de find.

J'aurais plutôt écrit ceci (en minimaliste) :

find /sys/devices/ -name modalias -exec cat '{}' \+

Qu'en pensez-vous ?
Je suis étonné devoir de telles commandes dans un man.

kamaris · Le 08/03/2023, à 13:10

Les gens privilégient souvent la lisibilité/simplicité, en estimant qu'un pipe et une substitution de commande sont plus lisibles/simples qu'une seule commande find (ou autre) ad hoc.
Un autre argument est que plus une commande est complexe, plus on va devoir se poser des questions sur sa portabilité, alors qu'un pipe et une substitution de commande, typiquement, ça marchera toujours.
D'autres arguments encore sont : négligence, fainéantise, manque de compétence

Tawal · Le 08/03/2023, à 13:56

Donc tu confirmerais que la bonne forme serait la mienne.

Analogie à ce que tu dis :
Si un jour mon maçon me dit : "Je fais le mur en moins bien mais ce sera plus joli et plus facile de savoir comment ça a été fait."
Je le fous dehors !

kamaris · Le 08/03/2023, à 14:20

> Donc tu confirmerais que la bonne forme serait la mienne.

En fait je pense que ce serait plutôt -name '*modalias*'

Watael · Le 08/03/2023, à 14:51

kamaris a écrit :

D'autres arguments encore sont : négligence, fainéantise, manque de compétence

on peut ajouter héritage de pratiques d'un autre âge.

tous les fichiers trouvés sont nommés modalias : aucun préfixe ni suffixe.

le grep est vraiment une erreur.
mais je ne vois pas de gain (sur 1000 itérations) entre la commande de Tawal et la première du man.

PS : le + n'étant pas un caractère spécial pour le shell il n'est pas indispensable de le protéger.
ça me fait encore bizarre de ne pas le protéger.

Dernière modification par Watael (Le 08/03/2023, à 14:53)

diesel · Le 08/03/2023, à 16:29

Tawal a écrit :

Donc tu confirmerais que la bonne forme serait la mienne.
Analogie à ce que tu dis :
Si un jour mon maçon me dit : "Je fais le mur en moins bien mais ce sera plus joli et plus facile de savoir comment ça a été fait."
Je le fous dehors !

A partir du moment où le résultat (affiché) est le même, les deux se valent (il n'est pas question de mur plus ou moins bien fait).

Donc, à résultat égal, j'aurais effectivement tendance (dans la mesure où il n'y a aucune problématique de temps d'exécution ou de consommation de ressources) à privilégier la forme la plus lisible et la plus portable.

Amicalement.

Jean-Marie

Dernière modification par diesel (Le 08/03/2023, à 16:30)

bruno · Le 08/03/2023, à 18:02

Watael a écrit :

kamaris a écrit :
D'autres arguments encore sont : négligence, fainéantise, manque de compétence
on peut ajouter héritage de pratiques d'un autre âge.

Comme ce genre de syntaxe :

if [ "x${GRUB_DISABLE_OS_PROBER}" = "xtrue" ]

Pour comprendre à lire en anglais

serged · Le 08/03/2023, à 18:24

Pour éviter les messages d'erreur avec "find" (ou d'autres commandes...), il faut renvoyer stderr sur /dev/null :

find [...] 2>/dev/null

kamaris · Le 08/03/2023, à 18:55

@bruno: excellent l'article, merci

Tawal · Le 09/03/2023, à 22:40

Oui, très bon article.

Ubuntu-fr

Navigation

Liens de recherche

Annonce

#1 Le 08/03/2023, à 12:16

Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#2 Le 08/03/2023, à 13:10

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#3 Le 08/03/2023, à 13:56

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#4 Le 08/03/2023, à 14:20

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#5 Le 08/03/2023, à 14:51

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#6 Le 08/03/2023, à 16:29

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#7 Le 08/03/2023, à 18:02

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#8 Le 08/03/2023, à 18:24

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#9 Le 08/03/2023, à 18:55

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

#10 Le 09/03/2023, à 22:40

Re : Bonne commande : cat $(find ...) ou find ... -exec cat '{}' \+

Pied de page des forums