[packagekit] error on remove/install

[Ken VanDine]

I don't think it will fix it... but will try.  This has been fine and
still works fine on Foresight 1.x, but Foresight 2 that we are working
on is built on a newer toolchain.  Not sure it is related, but this
still works fine in FL 1, just not in FL 2...  which is puzzling... same
source tarball...

I will pull and try.

--Ken

On Thu, 2007-11-08 at 18:41 +0000, Richard Hughes wrote:
> On Thu, 2007-11-08 at 11:40 -0500, Ken VanDine wrote:
> > [pk_engine_remove_package] pk-engine.c:1623 (11:38:31):  RemovePackage
> > method called: 7;dcbaebbe;data,
> > tmpwatch;2.9.10-2-2;x86;/conary.rpath.com at rpl:devel/2.9.10-2-2, 0
> > [pk_strvalidate] pk-common.c:216 (11:38:31):     invalid char in text!
> 
> Ahh, your package_id is tripping up the pk_strvalidate method - it
> thinks you're passing the method dangerous chars and hence refuses it.
> 
> static gboolean
> pk_strvalidate_char (gchar item)
> {
> 	switch (item) {
> 	case ' ':
> 	case '$':
> 	case '`':
> 	case '\'':
> 	case '"':
> 	case '^':
> 	case '[':
> 	case ']':
> 	case '{':
> 	case '}':
> 	case '@':
> 	case '#':
> 	case '\\':
> 	case '<':
> 	case '>':
> 	case '|':
> 		return FALSE;
> 	}
> 	return TRUE;
> }
> 
> So, your package_id is not valid because it contains an @ symbol, which
> is quite clearly a bug as you guys need this char. I've removed this
> char from the blacklist. Can you pull and try once again please? Thanks.
> 
> Richard.
> 
> 
> _______________________________________________
> PackageKit mailing list
> PackageKit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/packagekit