Hi again Didier,

On 3/5/21 11:53, Didier Kryn wrote:
     I'm very happy that you cath up on Hopman. I initially made it for
my needs and the community and as a proof of concept. Then I became too
ambitious and I didn't succeed to properly organize internationalization
files and I got discouraged. Yet the tool is working and it is my only
installed tool to mount hotplug USB disks.
I had a copy of your code, but i lost the part dedicated to the internationalization. It doesn't worry me because my approach in dealing with this matter will differ a bit from yours. However, i would like to verify i'm using your most recent code of Hopman anyway, and the gitlab repository, annoucced on the other hand in Emiliano Marini's website [*], no longer exists:

https://git.devuan.org/kryn/hopman <https://git.devuan.org/kryn/hopman>


     The first method I considered to determine which device was
hotpluggable was a config file made of a list of regular expressions to
match the device file names, such as "sd[c-f]*", untill I found that
kernel tag. Then I discovered, like you, that it only tagged USB
devices. But I sticked with the kernel tag by lazzyness and beause I
could live with it (I use an USB-SDcard adapter).


With the inclusion of some files taken from the vdev project, the device file names are no longer a problem to worry about. However, you'll be still able to list all the hotpluggable devices via:

$ lsblk -o PATH,HOTPLUG

The only requirement is util-linux. At  this point, as you already know, it's worth to highlight that *hotpluggable* is not the same as *removable*. *
*

*
*

     There is still a bug in a function in subdir watch, which I have
narrowed down to a logic error in the handling of multiple inotify
events in one buffer. This happens typically the first time a loop
device is mounted, because the driver then creates all loop devices at
once and the corresponding inotify events are small enough that more
than one can fit in the buffer; then hopman enters an endless loop.
Maybe I will dedicate time to repair it some day (~:

I'm not using inotify, but there is still an issue related to the use of lstat in the following file:

https://gitea.devuan.dev/aitor_czr/hopman/src/branch/master/hopman-1.0/watch/devpath.c <https://gitea.devuan.dev/aitor_czr/hopman/src/branch/master/hopman-1.0/watch/devpath.c>

only when you leave running hopman for an overly long time (which isn't the case in practice), because of the many opened files during the recursive scanning of "/sys/devices".

To end with, i've just updated the code in gitea including a screenshot, and let's say that it's working pretty fine:

https://gitea.devuan.dev/aitor_czr/hopman/src/branch/master <https://gitea.devuan.dev/aitor_czr/hopman/src/branch/master>

Btw, i added a reference in the headers of my files, and also included myself in the LICENSE file. The same with the code taken from vdev, mantaining Jude Nelson's copyright there. Is it right?

Cheers:)

Aitor.

[*] Aka linuxito:

https://www.linuxito.com/misc/1231-resumen-de-noticias-de-la-comunidad-devuan-agosto-2019 <https://www.linuxito.com/misc/1231-resumen-de-noticias-de-la-comunidad-devuan-agosto-2019>


_______________________________________________
Dng mailing list
Dng@lists.dyne.org
https://mailinglists.dyne.org/cgi-bin/mailman/listinfo/dng

Reply via email to