Error installing jabref deb package in ubuntu: native messaging host is not a dir

[Siedlerchr]

Latest version of JabRef: sudo dpkg -i jabref_5.0.0-1_amd64\(1\).deb

install: target '/usr/lib/mozilla/native-messaging-hosts/' is not a directory: No such file or directory
dpkg: error processing package jabref (--install):
 installed jabref package post-installation script subprocess returned error exit status 1
Errors were encountered while processing:
 jabref

I manually checked it, the dir is not existing. /usr/lib/mozilla only has a subfolder plugins

Linux osboxes 4.15.0-62-generic #69-Ubuntu SMP Wed Sep 4 20:55:53 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Ubuntu 18.04
Firefox 69

[LyzardKing]

 -D     create  all  leading  components of DEST except the last, or all
              components of --target-directory, then copy SOURCE to DEST

I think the issue is I used the destination, so it created only /usr/lib...
It should be enough to fix the postinst file...
I'll test now and upload a fix

[LyzardKing]

I created a branch (https://github.com/JabRef/jabref/tree/debRpmInstall) with the fix, so I can use it to also build and test the rpm before pushing to master.
For now if you want to test it the deb should work, so this fix can be merged.

[Siedlerchr]

Thank for the quick action, just create a PR and then we can checkout the branch build

[LyzardKing]

Done. If you want to test it before merging.. I tested on a freshly installed ubuntu-mate that did not have the mozilla folder before the install.
After a removal I only remove the json, although we could check IF the folder is empty and then remove that as well, but it might be unnecessary...

[Siedlerchr]

thanks works now. Only the "grep" is showing of course the dir not found, but it works:
grep: /usr/lib/mozilla/native-messaging-hosts/org.jabref.jabref.json: No such file or directory

[LyzardKing]

The grep should be from the postrm..
The check needs to be more consistent..
I think this can be solved by adding a check if the file exists..
I'll test it now.

[LyzardKing]

@Siedlerchr I pushed a fix on the same branch.
If you can test it once it builds let me know if it solves the issue

[Siedlerchr]

Will test it tomorrow on my VM (Reminder for myself)

[LyzardKing]

thanks @Siedlerchr
I rebased on master to be sure it's in sync...
So the latest version on the branch should be fine.
I guess that if it works for me and on your vm we can say it's fine ;)