Re: package-admin.el: hard coded '/' path separator

|--==> "APA" == Adrian Aichner <adrian(a)xemacs.org&gt; writes: Fabrice> (while path-list Fabrice> - (if (equal (substring (car path-list) -16) "xemacs-packages/") Fabrice> + (if (string-match "xemacs-packages[/\\]$" (car path-list)) Fabrice> (setq top-dir (car path-list))) APA> Steve Y, this patch may be better than using directory-sep-char. APA> On windows people mix / and \\ a lot and they both work. Ew! How obtuse! Does this mean that 'directory-sep-char' won't work here? That just plain sucks. I'll never understand why we support that poor excuse for an OS anyway. Wouldn't the best fix be to fix 'directory-sep-char' so it can handle this type of bogosity? I don't mind using Fabrice's patch in the interim, if that's what's needed. APA> We should also handle the case more gracefully (easy to APA> understand error message) where the package path does not APA> contain mule-packages or xemacs-packages at all (think APA> Debian). Improvements could probably be made in this regard, yes. Patches welcome.:-) APA> Just my 0.02~ Does that include GST? -- |---<Steve Youngs>---------------<GnuPG KeyID: 10D5C9C5>---| | XEmacs - It's not just an editor. | | It's a way of life. | |------------------------------------&lt;youngs(a)xemacs.org&gt;---|