From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann E. MORIN Date: Fri, 31 Jul 2015 00:40:37 +0200 Subject: [Buildroot] [PATCH 2/3] package/xserver: commonalise dependency on libdrm In-Reply-To: <20150729225257.1dbe308d@free-electrons.com> References: <4368a6cd5cf0f903bc2e0dd756b413b9ed8378ad.1437950027.git.yann.morin.1998@free.fr> <20150729225257.1dbe308d@free-electrons.com> Message-ID: <20150730224037.GG3744@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Thomas, All, On 2015-07-29 22:52 +0200, Thomas Petazzoni spake thusly: > On Mon, 27 Jul 2015 00:35:11 +0200, Yann E. MORIN wrote: > > +XSERVER_XORG_SERVER_NEEDS_LIBDRM = y > Shouldn't we use an hidden Config.in boolean instead for this? OK, I looked at it, and althogh I saif I would do it, I would like to avoid doing it. The reason is that the conditions under which libdrm is needed ar ealready known in the .mk, but not in the Config.in. So, I would find it a bit inconvenient to have to duplicate that knowledge it two locations. In the end, I believe the current patch is the best we can get. Regards, Yann E. MORIN. -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------'