<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 2 mai 2022 à 12:24, Jonas Smedegaard <<a href="mailto:jonas@jones.dk">jonas@jones.dk</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Quoting Jérémy Lal (2022-05-02 11:59:15)<br>
> Le lun. 2 mai 2022 à 11:54, Jonas Smedegaard <<a href="mailto:jonas@jones.dk" target="_blank">jonas@jones.dk</a>> a écrit :<br>
> <br>
> > Quoting Jérémy Lal (2022-05-02 11:45:36)<br>
> > > I stand by saying as it is, putting those packages:<br>
> > > node-chalk<br>
> > > node-strip-ansi<br>
> > > node-text-table<br>
> > > in Recommends just breaks the default functionality of eslint.<br>
> ><br>
> > Ignoring recommends breaks systems.<br>
> ><br>
> <br>
> eslint is a build-dependency of enigmail (nothing odd about that).<br>
> When sbuild builds a package, it doesn't install recommended packages <br>
> of build-dependencies ?<br>
<br>
Correct: Build environments need recommendations explicitly declared as <br>
build-dependencies, when needed.<br></blockquote><div><br></div><div>I don't discuss the usefulness of Recommends.</div><div><br></div><div>I'm trying to argue why in the particular case of eslint and of those three packages</div><div>(mentioned above) you put in Recommends, it breaks eslint for other packages</div><div>Build-Depending on eslint.</div><div><br></div><div>Adding those Recommends in Build-Depends is not a good practice, IMO,</div><div>for this reasons:</div><div>- separation of concerns</div><div>- Build-Depends eslint does not install Recommends resulting in a broken eslint</div><div>- when some eslint recommends change, build-dependent packages will just break again !</div><div>- worse (because undetectable) build-dependent packages will uselessly build-depend on</div><div>no longer required packages.</div><div><br></div><div>Is there a strong reason to put those three packages in Recommends ?</div><div>The economy of installing three additional small packages is outweighed by the</div><div>over-engineering needed to cope with it.</div><div><br></div><div>Jérémy</div></div></div>