On 09/14/2017 08:09 AM, Thomas Lamprecht wrote: > On 09/13/2017 04:10 PM, Emmanuel Kasper wrote: >> This allows calling the 'make install' target twice in a row. > > Looks OK, we overwrite everything else so why not the man pages. > > This can be already applied even if we would say the current > iteration of this series still lacks something, IMO, no need > to drag this along... > > Also, FYI: I assume you made this patch in the first place to > allow faster testing your changes by just doing `make install` > circumventing the slower package build of `make dinstall` where > this patch wouldn't be needed? > If so, you could like to use perl's -I switch: > >> -Idirectory >> Directories specified by -I are prepended to the search >> path for modules (@INC). > > > For example, in your case you could do the following from inside the > qemu-servers top level directory: > > perl -I. ./qm importovf ... > > Just for the case that you did not know this all already :) >
I was using use lib qw(..); in the unit tests, but not the -I switch for the command line thanks for the hint _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
