Created by: darko-poljak
@matthijskooijman Please test fix: 23292e5cad
Created by: matthijskooijman
Fix looks good to me and works, thanks!
Created by: darko-poljak
@matthijskooijman Inventory location defaults to ~/.cdist/inventory because there is nothing related to inventory that comes with distribution, except implementation…
Created by: darko-poljak
@matthijskooijman Can you test the following commit, related to empty HOME env var? 2b5f1a0148
Created by: matthijskooijman
Works, thanks.
As for the in-distribution location, let me explain my usecase. I'm using cdist to configure a number of similar machines. For this, I forked the…
Created by: darko-poljak
@matthijskooijman You can also use configuration file: https://www.nico.schottelius.org/software/cdist/man/latest/cdist-configuration.html There is an option for…
Created by: matthijskooijman
Yeah, that's also an option. I still need to pass a -g option explicitly. Would it make sense to automatically load a config file from the distribution, if it exists?…
Created by: darko-poljak
@matthijskooijman You can have user's configuration file (first one found of ~/.cdist.cfg, $XDG_CONFIG_HOME/cdist/cdist.cfg, in specified order) or system-wide…
Created by: matthijskooijman
Yeah, but none of these are inside my repository, so I can't make the repository self-contained (and it also won't work if I want to work with multiple self-contained…
Created by: darko-poljak
How about ln -s <file inside repo> ~/.cdist.cfg ?
Created by: matthijskooijman
That would work, except that:
- This requires users to modify their environment after cloning the repo (with the risk of users forgetting to make the link, or later…
Created by: darko-poljak
@matthijskooijman @telmich I think we can take in-distribution config file into account if it exists.
Created by: matthijskooijman
A bit late, but: Thanks! Works like a charm for me :-)
Created by: darko-poljak
@matthijskooijman Please check 3fb7e33305
Created by: darko-poljak
And I thought we are ready for new bugfix release.
Created by: darko-poljak
@telmich Can you try with latest master version? This should have ben fixed in https://github.com/ungleich/cdist/issues/639. The latest version is cdist…