From owner-svn-src-all@FreeBSD.ORG Thu Apr 30 19:40:37 2015 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 1D78FEF1; Thu, 30 Apr 2015 19:40:37 +0000 (UTC) Received: from mailrelay119.isp.belgacom.be (mailrelay119.isp.belgacom.be [195.238.20.146]) by mx1.freebsd.org (Postfix) with ESMTP id B176A1324; Thu, 30 Apr 2015 19:40:35 +0000 (UTC) X-Belgacom-Dynamic: yes X-Cloudmark-SP-Filtered: true X-Cloudmark-SP-Result: v=1.1 cv=wisUSL2q+2X910teRg2xg/30tBFW7lBMAi45pEFXNHU= c=1 sm=2 a=6I5d2MoRAAAA:8 a=uZvujYp8AAAA:8 a=l6kVda_T1Z9zCi4Qi1MA:9 a=CjuIK1q_8ugA:10 a=tKbmi1Y-NooA:10 a=tNJVa7PJUT1K0kJ5Z0MA:9 a=dmpdFviX8IwA:10 a=0ZwACUlkkugA:10 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: A2CZBgCohEJV/wu/8VFcgwxTUAzFWYFThTZOAoFWOhMBAQEBAQEBgQqEIQEBBFYjEAsOBgQJGgsPKh4GE4gvAQjIYgEBAQEBAQQBAQEBAQEBARqLOIR0EQeELQWPNYY+hj6BYYZgg3KJSyOCOIE+PDEBgkQBAQE Received: from 11.191-241-81.adsl-dyn.isp.belgacom.be (HELO kalimero.tijl.coosemans.org) ([81.241.191.11]) by relay.skynet.be with ESMTP; 30 Apr 2015 21:39:09 +0200 Received: from kalimero.tijl.coosemans.org (kalimero.tijl.coosemans.org [127.0.0.1]) by kalimero.tijl.coosemans.org (8.14.9/8.14.9) with ESMTP id t3UJd7Xe004974; Thu, 30 Apr 2015 21:39:07 +0200 (CEST) (envelope-from tijl@FreeBSD.org) Date: Thu, 30 Apr 2015 21:39:02 +0200 From: Tijl Coosemans To: Bryan Drewery Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org Subject: Re: svn commit: r282275 - in stable/10: . bin/csh contrib/smbfs/include/netsmb contrib/smbfs/lib/smb include lib/libarchive lib/libc/iconv lib/libc/locale lib/libiconv_modules/BIG5 lib/libiconv_modules... Message-ID: <20150430213902.21ee48fc@kalimero.tijl.coosemans.org> In-Reply-To: <55427802.80300@FreeBSD.org> References: <201504301608.t3UG8mNS004104@svn.freebsd.org> <55427802.80300@FreeBSD.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; boundary="Sig_/6klm17pZzuaratc4hqX+at0"; protocol="application/pgp-signature" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 30 Apr 2015 19:40:37 -0000 --Sig_/6klm17pZzuaratc4hqX+at0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Thu, 30 Apr 2015 13:44:18 -0500 Bryan Drewery wro= te: > On 4/30/2015 11:08 AM, Tijl Coosemans wrote: >> Author: tijl >> Date: Thu Apr 30 16:08:47 2015 >> New Revision: 282275 >> URL: https://svnweb.freebsd.org/changeset/base/282275 >>=20 >> Log: >> MFC r275805: >> =20 >> Fix incorrect type of "invalids" argument in __iconv() prototype. >> =20 >> MFC r281550,281591: >> =20 >> Remove the const qualifier from iconv(3) to comply with POSIX: >> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iconv.html >> =20 >> Adjust all code that calls iconv. >> =20 >> PR: 199099 >>=20 >> Modified: >> stable/10/UPDATING >> stable/10/bin/csh/config.h >> stable/10/bin/csh/iconv_stub.h >> stable/10/contrib/smbfs/include/netsmb/smb_lib.h >> stable/10/contrib/smbfs/lib/smb/nls.c >> stable/10/contrib/smbfs/lib/smb/print.c >> stable/10/contrib/smbfs/lib/smb/rq.c >> stable/10/include/iconv.h >> stable/10/lib/libarchive/Makefile >> stable/10/lib/libc/iconv/__iconv.c >> stable/10/lib/libc/iconv/bsd_iconv.c >> stable/10/lib/libc/iconv/citrus_iconv.h >> stable/10/lib/libc/iconv/citrus_iconv_local.h >> stable/10/lib/libc/iconv/citrus_none.c >> stable/10/lib/libc/iconv/citrus_stdenc.h >> stable/10/lib/libc/iconv/citrus_stdenc_local.h >> stable/10/lib/libc/iconv/citrus_stdenc_template.h >> stable/10/lib/libc/iconv/iconv-internal.h >> stable/10/lib/libc/iconv/iconv.3 >> stable/10/lib/libc/iconv/iconv.c >> stable/10/lib/libc/iconv/iconv_compat.c >> stable/10/lib/libc/locale/cXXrtomb_iconv.h >> stable/10/lib/libc/locale/mbrtocXX_iconv.h >> stable/10/lib/libiconv_modules/BIG5/citrus_big5.c >> stable/10/lib/libiconv_modules/DECHanyu/citrus_dechanyu.c >> stable/10/lib/libiconv_modules/EUC/citrus_euc.c >> stable/10/lib/libiconv_modules/EUCTW/citrus_euctw.c >> stable/10/lib/libiconv_modules/GBK2K/citrus_gbk2k.c >> stable/10/lib/libiconv_modules/HZ/citrus_hz.c >> stable/10/lib/libiconv_modules/ISO2022/citrus_iso2022.c >> stable/10/lib/libiconv_modules/JOHAB/citrus_johab.c >> stable/10/lib/libiconv_modules/MSKanji/citrus_mskanji.c >> stable/10/lib/libiconv_modules/UES/citrus_ues.c >> stable/10/lib/libiconv_modules/UTF1632/citrus_utf1632.c >> stable/10/lib/libiconv_modules/UTF7/citrus_utf7.c >> stable/10/lib/libiconv_modules/UTF8/citrus_utf8.c >> stable/10/lib/libiconv_modules/VIQR/citrus_viqr.c >> stable/10/lib/libiconv_modules/ZW/citrus_zw.c >> stable/10/lib/libiconv_modules/iconv_none/citrus_iconv_none.c >> stable/10/lib/libiconv_modules/iconv_std/citrus_iconv_std.c >> stable/10/lib/libkiconv/xlat16_iconv.c >> stable/10/sys/sys/param.h >> stable/10/usr.bin/iconv/iconv.c >> Directory Properties: >> stable/10/ (props changed) >>=20 >> Modified: stable/10/UPDATING >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D >> --- stable/10/UPDATING Thu Apr 30 15:48:48 2015 (r282274) >> +++ stable/10/UPDATING Thu Apr 30 16:08:47 2015 (r282275) >> @@ -16,6 +16,10 @@ from older versions of FreeBSD, try WITH >> stable/10, and then rebuild without this option. The bootstrap process = from >> older version of current is a bit fragile. >> =20 >> +20150430: >> + The const qualifier has been removed from iconv(3) to comply with >> + POSIX. The ports tree is aware of this from r384038 onwards. >> + >=20 > Being an ABI change this seems wrong to MFC. Binaries are not only built > from Ports. >=20 > For a binary built on 10.1 will this change cause any issues? Removing const from a function parameter is not an ABI change, so there are no issues for binaries. The worst you can run into is a warning/error from a compiler (when you pass a const char** argument to what is now a char** parameter), but this should be trivial to fix. --Sig_/6klm17pZzuaratc4hqX+at0 Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iF4EAREKAAYFAlVChNYACgkQfoCS2CCgtitKxwD/bMbwoaX6u5LAOUif8z78FLAR k5rIvzRZDcymResjj28A/iNkn6c+7Dabj1y8VpFSBthz4rDhMyZ+IYiCLIowFM+x =mktF -----END PGP SIGNATURE----- --Sig_/6klm17pZzuaratc4hqX+at0--