Date: Fri, 7 Sep 2018 18:00:17 +0200 From: Baptiste Daroussin <bapt@FreeBSD.org> To: Fernando =?utf-8?Q?Apestegu=C3=ADa?= <fernape@FreeBSD.org> Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r479196 - head/sysutils/smartmontools/files Message-ID: <20180907160017.r5vf67bnkdqgkwdx@ivaldir.net> In-Reply-To: <201809071545.w87FjEBE098699@repo.freebsd.org> References: <201809071545.w87FjEBE098699@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--ojhspkadkphe4q3y Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Sep 07, 2018 at 03:45:14PM +0000, Fernando Apestegu=EDa wrote: > Author: fernape > Date: Fri Sep 7 15:45:14 2018 > New Revision: 479196 > URL: https://svnweb.freebsd.org/changeset/ports/479196 >=20 > Log: > sysutils/smartmontools: unbreak in 12-CURRENT > =20 > Between __FreeBSD_version 1200058 and 1200081 fields opc and fuse of > struct nvme_command (sys/dev/nvme/nvme.h) where merged under opc_fuse, = but > in r338182 this change was undone and this port broke. > =20 > Update patch to fix it so it works regardless of the revision we check = out. > =20 > PR: 230867 > Submitted by: ohartmann@walstatt.org > Approved by: tcberner (mentor), maintainer (timeout, 2 weeks) > Differential Revision: https://reviews.freebsd.org/D16908 >=20 > Modified: > head/sysutils/smartmontools/files/patch-os__freebsd.cpp >=20 > Modified: head/sysutils/smartmontools/files/patch-os__freebsd.cpp > =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 > --- head/sysutils/smartmontools/files/patch-os__freebsd.cpp Fri Sep 7 14= :47:48 2018 (r479195) > +++ head/sysutils/smartmontools/files/patch-os__freebsd.cpp Fri Sep 7 15= :45:14 2018 (r479196) > @@ -22,7 +22,7 @@ > struct nvme_pt_command pt; > memset(&pt, 0, sizeof(pt)); > =20 > -+#if __FreeBSD_version >=3D 1200058 > ++#if __FreeBSD_version >=3D 1200058 && __FreeBSD_version < 1200081 > + pt.cmd.opc_fuse =3D NVME_CMD_SET_OPC(in.opcode); > +#else > pt.cmd.opc =3D in.opcode; >=20 Is it worth keeping a patch for just a very tiny set of revision of -HEAD? Best regards, Bapt --ojhspkadkphe4q3y Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEgOTj3suS2urGXVU3Y4mL3PG3PloFAluSoHgACgkQY4mL3PG3 PlrsmA/9ECrYbkE3o+S1X11MWQKM7sFiZg54PnIks/3YoDa1W5oOjC55cwaE9d2K YiEstuSyjolvqqUSsEaWbMyf7L9ObiE+TgiYjKxX6aCILvqIKsPWi2yCpgCp1pG6 Xtm93BNCzgs5ttVInzFk+0LNKYIwPA9BDS+ESSU+hAwREXEidM/RQJc9Vl32sYrP m8FEj3dqxwERx3ePsTkRnTxMKUF3USDoQMHvJvkrh8vVVImT89bUYfMhkWZzaq8h vqZO1yF9iwvZ9xTpxal7xdB/lQTxIIruWp78uKtR/uNxP130hUvRPnWSUvCgFxfr NYI/K1lEFVOaMm1QCSreTopVOIhnEpJ7DIE0zQznepV/TzVQzsgKvLw9JUrGSbZf OALu7aB099Qx64s2xluKOw7u96n2MrUThHpZBC/O1b6lxrFgGvXx1IsxsC+iDOAI XYBi1y49gv7eNeig5daENGDZCKBEXJzaVrcnRimA3q3B/HdvGzPdEF48NYd2P59h Gotk68PUKvq1JQusm2rGN0omSvU+pXrb/VyCkcnkjtOiNmGZe894LeJqkU6LrECs X6toyZvk49HkOv8xjKdbPJ2TzhjYZFxpzS5BizaKloM9id3/odt+SsqOqCV11N4o aa6LDv4h2Pm2GjsuGRpgh612ATXwOEYBitbfQP6WEmNRYLpDcd8= =zwBQ -----END PGP SIGNATURE----- --ojhspkadkphe4q3y--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180907160017.r5vf67bnkdqgkwdx>