Date: Sat, 24 Oct 2009 02:35:42 -0400 From: Joe Marcus Clarke <marcus@marcuscom.com> To: "J.R. Oldroyd" <fbsd@opal.com> Cc: freebsd-gnome@freebsd.org Subject: Re: hal truncating block.device on freebsd-8.0rc1 Message-ID: <1256366142.54447.188.camel@shumai.marcuscom.com> In-Reply-To: <20091023141921.7aaecf23@shibato.opal.com> References: <20091022212532.23a0b972@shibato.opal.com> <1256270523.54447.113.camel@shumai.marcuscom.com> <20091023013730.6300fd64@shibato.opal.com> <1256276672.54447.130.camel@shumai.marcuscom.com> <20091023141921.7aaecf23@shibato.opal.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--=-v0Kpzg7aQBu/K/8Hv4vO Content-Type: text/plain Content-Transfer-Encoding: quoted-printable On Fri, 2009-10-23 at 14:19 -0400, J.R. Oldroyd wrote: > On Fri, 23 Oct 2009 01:44:32 -0400, Joe Marcus Clarke <marcus@marcuscom.c= om> wrote: > > > > On Fri, 2009-10-23 at 01:37 -0400, J.R. Oldroyd wrote: > > > On Fri, 23 Oct 2009 00:02:03 -0400, Joe Marcus Clarke <marcus@marcusc= om.com> wrote: > > > >=20 > > > > Hal doesn't support volumes with spaces in their names. This is > > > > documented in the FreeBSD HAL FAQ at > > > > http://www.freebsd.org/gnome/docs/halfaq.html . Patches to add sup= port > > > > for this are very welcome. > > > >=20 > > > > Joe > > > >=20 > > >=20 > > > That doc, and some googling which found two year old email from you, > > > show that this is apparently a long-standing problem with geom/glabel= . > > >=20 > > > Any pointers on where the defective code might lie? > >=20 > > hald/freebsd/hf-storage.c. The fix would be to process > > kern.geom.confxml instead of kern.geom.conftxt. > >=20 > > Joe > >=20 >=20 > I have a simple patch to the conftxt parser that looks for LABEL > lines and appends fields until the "i" field is in the right > place. This fixes things for this camera. >=20 > I have no other devices at hand to test to see if anything broke, > however. >=20 > The replacement files/patch-hald_hf-storage.c with the additional > patch is here: > http://opal.com/jr/hal/patch-hald_hf-storage.c Thanks. I believe this will work given that the format of conftxt looks to be consistent across 6.X, 7.X, 8.X, and 9.X. I've modified the patch some for safety and style, and added it to hal-0.5.13 which will be committed with GNOME 2.28 when 8.0 is released. Joe >=20 > By the way, the patch file should probably be renamed to > files/patch-hald_freebsd_hf-storage.c for consistency with the > other patch files there. >=20 > -jr >=20 --=20 PGP Key : http://www.marcuscom.com/pgp.asc --=-v0Kpzg7aQBu/K/8Hv4vO Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (FreeBSD) iEYEABECAAYFAkrioDwACgkQb2iPiv4Uz4caNQCfR+T6+plsRNapJdmWaq3oGaw+ xlUAn3bPXRIJjIXJTM2UibReNf0p0v9Y =Rp6r -----END PGP SIGNATURE----- --=-v0Kpzg7aQBu/K/8Hv4vO--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1256366142.54447.188.camel>