[freedict] discussion: make install and other FD improvements

  • From: Einhard Leichtfuß <alguien@xxxxxxxxxxxxx>
  • To: freedict@xxxxxxxxxxxxx
  • Date: Tue, 1 May 2018 14:51:18 +0200

Sorry for the late reply. Not that much time the last weeks.

On 2018-04-03 11:27, Sebastian Humenda wrote:

Hi all

A few of the things below discuss points of our infrastructure and especially 
on
the `make install` rule, I would like to hear your comments: have you ever 
used
it, what are your feelings about it?
Einhard Leichtfuß wrote:
A note on the Makefile (tools/mk/dicts.mk)
- As noted above, the install target attempts to restart dictd. I do not
 consider that a good idea. Particularly, it won't have the desired
 effect (on Arch at least), as restarting dictd does not add the
 dictionary files to dictd.conf.
What makes you beleive that only the dict server is restarted? `dictdconfig` 
is
run as well, required for dictd to learn about new databases.

As you noted in another e-mail, the install target does not restart
dictd, but the install-restart target does. I must have been confused by
the echo statements in the install target. Hence I do not really have a
problem with it.

On Arch at least, dictdconfig is not available. If I recall correctly, I
was once told it would be debian specific.

 Also, there may be reasons not to restart dictd, e.g. when installing
 many dictionaries at a time, since restarting dictd (at least using
 systemd) several times in a row may make it fail.
I'm not a big fan of this script anyway. Debian simply doesn't use it and it 
was
only kept because Michael deemed it useful for development. Is it a problem to
not use `make install` on Arch? I'm also open for removing the make install 
rule
all together, but it's nice to have it, as I said, for ddevelopment purposes.

I'm fine writing a few installation statements on my own.

If we should keep it, could you provide a patch to make it work on Arch as 
well?
I've tried to make it work on multiple distributions, without actual knowledge
of the other environments.

In fact, the install target should just work as it is. Maybe I'll change
my PGBUILDs one day, though that would not really make them more readable.

- Also, I'd like documentation files being copied by `make install'. I
 currently do not use `make install' in my packages, but instead
 manually copy the files, including any of AUTHORS, README, NEWS,
 ChangeLog, if existing.
IMHO that is the task of a package maintainer :), but maybe you have a 
different
opinion on that. Where would you install these files to? Dictd usually wants 
it
dictionaries under /usr/share/dictd on Debian, a place where the doc files
shouldn't go to, they go under /usr/share/doc/<PKGNAME>/doc. But I reckon this
is Debian-specific.

I do not really know about common documentation places, except on Arch,
where it's /usr/share/doc/<PKGNAME>. Regarding the differen place on
Debian, you're probably right.

- Checksums are great. Maybe you could provide such in the API?
 Currently, my script fetches the sources / binaries on every update,
 just to calculate the checksum.
Brilliant, what do you want, MD5?

Since there should not be a security impact (is there one?), MD5 should
be fine. Though I generally prefer stronger ones, such as SHA512.

- In the "list of clients" on GitHub, I miss the mention of `dict',
 being part of dictd itself, as far as I know. In my eyes, the most
Fixed.

Thanks.

 Also, the XFCE4 dictionary [0] may be considered worth including.
Could you please add this? If you give me your user name (PM is fine), I'll 
add
you as a collaborator.

respiranto.

Greetings,
Einhard

Attachment: signature.asc
Description: OpenPGP digital signature

Other related posts:

  1. Home
  2. freedict
  3. Archive
  4. 05-2018