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

SY> |--==> "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. SY> Ew! How obtuse! Does this mean that 'directory-sep-char' won't work SY> here? That just plain sucks. I'll never understand why we support SY> that poor excuse for an OS anyway. SY> Wouldn't the best fix be to fix 'directory-sep-char' so it can handle SY> this type of bogosity? I don't mind using Fabrice's patch in the SY> interim, if that's what's needed. Don't think so since 'directory-sep-char' is just a char. 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). SY> Improvements could probably be made in this regard, yes. Patches SY> welcome.:-) Yep, I know. APA> Just my 0.02~ SY> Does that include GST? What's GST? SY> -- SY> |---<Steve Youngs>---------------<GnuPG KeyID: 10D5C9C5>---| SY> | XEmacs - It's not just an editor. | SY> | It's a way of life. | SY> |------------------------------------&lt;youngs(a)xemacs.org&gt;---| -- Adrian Aichner mailto:adrian＠xemacs.org http://www.xemacs.org/