D'oh!
You are perfectly right, shame on me.

I sort of manually created the patch and it seems I didn't include all of it.
The line you linked to has to be changed to "zip_file.extractall(local_dir)" 
because my patch already adds the correct plugin dir to the local_dir 
variable.





Am Dienstag, 6. März 2018, 17:09:57 CET schrieb Yann Leboulanger:
> That doesn't work. In this case it downloads plusins in
> ~/.local/gajim/plugins.PROFILE_NAME/plugins instead of directly in
> ~/.local/gajim/plugins.PROFILE_NAME.
> 
> This is because of
> https://dev.gajim.org/gajim/gajim-plugins/blob/master/plugin_installer/plugi
> n_installer.py#L541
> > In gajim master branch separate plugin directories per profile doesn't
> > work as expected anymore (one cannot install plugins anymore).
> > This small patch fixes the problem:
> > 
> > --- -       2018-03-04 02:04:46.779618319 +0100
> > +++ /usr/lib/python3/dist-packages/gajim/data/plugins/plugin_installer/
> > plugin_installer.py 2018-03-04 02:04:43.713257743 +0100
> > @@ -528,7 +528,7 @@
> > 
> >               local_dir = os.path.join(user_dir, remote_dir)
> >               
> >               if not os.path.isdir(local_dir):
> >                   os.mkdir(local_dir)
> > 
> > -            local_dir = os.path.split(user_dir)[0]
> > +            local_dir = os.path.dirname(local_dir)
> > 
> >               # downloading zip file
> > 
> >               try:
> > _______________________________________________
> > Gajim-devel mailing list
> > Gajim-devel@gajim.org
> > https://lists.gajim.org/cgi-bin/listinfo/gajim-devel
> 
> _______________________________________________
> Gajim-devel mailing list
> Gajim-devel@gajim.org
> https://lists.gajim.org/cgi-bin/listinfo/gajim-devel
_______________________________________________
Gajim-devel mailing list
Gajim-devel@gajim.org
https://lists.gajim.org/cgi-bin/listinfo/gajim-devel

Reply via email to