From mboxrd@z Thu Jan 1 00:00:00 1970 From: Doug Goldstein Subject: Re: [PATCH] build: add --with-rundir option to configure Date: Fri, 24 Feb 2017 10:49:16 -0600 Message-ID: References: <20170216074707.27135-1-jgross@suse.com> <20170220143145.mkstyadebue2bovy@citrix.com> <138d4c6a-ace9-1818-03b2-6a631bc6ceeb@suse.com> <96342cbd-ba1c-bb06-229a-5e03000571e3@suse.com> <9b3ed11f-a455-0c0c-f9c5-61521445e723@cardoe.com> <481dfba0-9213-b1e3-c4b8-c14e888ecd9a@suse.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============3874171630774711300==" Return-path: Received: from mail6.bemta6.messagelabs.com ([193.109.254.103]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1chJ3q-0003VT-Rz for xen-devel@lists.xenproject.org; Fri, 24 Feb 2017 16:49:39 +0000 Received: by mail-yb0-f193.google.com with SMTP id a5so974204ybb.3 for ; Fri, 24 Feb 2017 08:49:35 -0800 (PST) In-Reply-To: <481dfba0-9213-b1e3-c4b8-c14e888ecd9a@suse.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Juergen Gross , Andrew Cooper , Wei Liu Cc: xen-devel@lists.xenproject.org, ian.jackson@eu.citrix.com List-Id: xen-devel@lists.xenproject.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --===============3874171630774711300== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="QdkqAh45OSBTbrFa2F1Mhwajq4bVV9Bex" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --QdkqAh45OSBTbrFa2F1Mhwajq4bVV9Bex Content-Type: multipart/mixed; boundary="XeXkxXMTuTldv8s5WTTpQeXvRB7f9BFW9"; protected-headers="v1" From: Doug Goldstein To: Juergen Gross , Andrew Cooper , Wei Liu Cc: xen-devel@lists.xenproject.org, ian.jackson@eu.citrix.com Message-ID: Subject: Re: [Xen-devel] [PATCH] build: add --with-rundir option to configure References: <20170216074707.27135-1-jgross@suse.com> <20170220143145.mkstyadebue2bovy@citrix.com> <138d4c6a-ace9-1818-03b2-6a631bc6ceeb@suse.com> <96342cbd-ba1c-bb06-229a-5e03000571e3@suse.com> <9b3ed11f-a455-0c0c-f9c5-61521445e723@cardoe.com> <481dfba0-9213-b1e3-c4b8-c14e888ecd9a@suse.com> In-Reply-To: <481dfba0-9213-b1e3-c4b8-c14e888ecd9a@suse.com> --XeXkxXMTuTldv8s5WTTpQeXvRB7f9BFW9 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 2/24/17 10:14 AM, Juergen Gross wrote: > On 24/02/17 17:06, Doug Goldstein wrote: >> On 2/22/17 1:53 AM, Juergen Gross wrote: >>> On 20/02/17 16:19, Andrew Cooper wrote: >>>> On 20/02/17 14:43, Juergen Gross wrote: >>>>> On 20/02/17 15:31, Wei Liu wrote: >>>>>> On Thu, Feb 16, 2017 at 08:47:07AM +0100, Juergen Gross wrote: >>>>>>> There have been reports that Fedora 25 uses /run instead of /var/= run. >>>>>>> >>>>>>> Add a --with-rundir option ito configure to be able to specify th= at >>>>>> I've read this thread but I'm not sure if I need to take any actio= n or >>>>>> all the comments addressed -- especially the part about autoconf. >>>>> Andrew, are you fine with my answer regarding autoconf? Or do you h= ave >>>>> some information regarding --runstatedir which could help? >>>> Oh sorry. Didn't realise I was blocking here. I have no specific >>>> information, other than the quick search I did. >>>> >>>> Can't the future problem be worked around just with if autoconf vers= ion >>>> < 2.70 ? >>> I don't think it is possible to add configure options other than >>> --disable-*, --enable-*, --with-* or --without-* by other means than >>> patching general.m4 of autoconf. I don't think we want to do that. >>> >>> So the possibilities are: >>> >>> 1. don't support /run instead of /var/run via configure >>> 2. patch autoconf to support --runstatedir >>> 3. take this patch adding support via --with-rundir and possibly >>> switch over to --runstatedir when a new autoconf version is >>> available >>> >>> I'm in favor of (3.). >>> >>> >>> Juergen >> FWIW, many distros have already pulled the patch into their autoconf s= o >> its available so you wouldn't really have to do anything. >> >> Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=3D759647 >> Ubuntu: 16.04 and newer have it >> Gentoo: no link handy but I know its there >=20 > openSUSE: not available >=20 > I don't think its a good idea to rely on _all_ relevant distributions > having done the backport. >=20 >=20 > Juergen >=20 I was under the impression that patches against the configure script are just to the .ac file and the committer is responsible to regenerate the configure script that's committed and its suppose to happen on a Debian machine. But that being said I would suggest that openSUSE pull in the patch as we= ll. And I'm also not trying to convince people to not go with option 3. Feel free to ignore me as noise. --=20 Doug Goldstein --XeXkxXMTuTldv8s5WTTpQeXvRB7f9BFW9-- --QdkqAh45OSBTbrFa2F1Mhwajq4bVV9Bex Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0 iQJ8BAEBCgBmBQJYsGQPXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRBNTM5MEQ2RTNFMTkyNzlCNzVDMzIwOTVB MkJDMDNEQzg3RUQxQkQ0AAoJEKK8A9yH7RvUIQcP/03ZCPLGVVHtvImHzEF9H/p0 JSP1zMB1kYqrz9FaC6LPaAhL3rgCFOjKMEvYvwi4wZGNIQyHIhBKFBG8HAP3yqye SyICAngW/b6Sb+j/DNa4Vq2IpFstQOnfKtSkqZ23gWyi+bQ3eGZ6LzsyW92Qyg0O qOov1cNT0icuBxkbNoGXLufwtO04h5FxIevFEJaQq0YItYbhxZD3KgcEn9Mp2LHi zcYvSjWjAZ31q6QRpM6kriaIK+s9xrh15JTWW3Hj+vfls68Q0WytA3pAQoY7meZm U9ZLH1WRNUxaYa3a+b7HnLFHWh3ANbDlPwZbiyg8JBlqdg2YKf1pgnh/5ThvWbLm mwunC6o5nTe08qNgco3ZdaaDojENOwdjm2L46iqNi9pVYt8KZpwJnEzaEw5EzunX hntxwJ4zrXnE7XpvRw1+fJRfNlhml4/nJHXzw2R65WN3WQG5n3I5o2xnyoAwvdd1 PWMWc6Vgb9RD6X8Dr7wYx7SgSBlUP3NBAmWkyEYb016YiwPU5ZbcjvBlFNtUWKOt oL4vW31zaygDSShnsbsfHdBzO4b+HbE2PcJJyipjEdxATj5wvq2IB2bZ0NTexYq0 motSLUMf0GUGdX9NTqBDVVkbe3BLqVAosMA68fwcIsJvFyQPEEBaV2Dv03f6f5YW ZV3PJsD276coyH9J2pCS =PCDU -----END PGP SIGNATURE----- --QdkqAh45OSBTbrFa2F1Mhwajq4bVV9Bex-- --===============3874171630774711300== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwczovL2xpc3RzLnhlbi5v cmcveGVuLWRldmVsCg== --===============3874171630774711300==--