Robert Chéramy

Results 218 comments of Robert Chéramy

This is an uncommon use case for me, but if you made the effort of submitting a PR, it is worth integrating it into oxidized. Thanks for your PR! The...

Notes: - This may imply to merge scp.rb into ssh.rb, see input from @ytti in https://github.com/ytti/oxidized/commit/0bfddc0f0804faf3d9bfbe8ae8c0ee694ce99257#r162861892 - I'm thinking of adding `download` oder `get_file` in parallel to `cmd` to differentiate...

@systeembeheerder - if this works for you, I'll merge into master.

This is a pattern used some models, at least for unsupported commands (https://github.com/ytti/oxidized/blob/master/lib/oxidized/model/aosw.rb#L57) But I agree that this case is different as the command is supported, only the output of...

> Note - `cfg = cfg.cut_both if screenscrape` does not seam to work correctly. resulting in the prompt being spread through the output. But this is not the topic of...

I'm not sure how to deal with this PR. My impression is that the discussion between @ytti and @Filippo125 is not finished. If this PR has to be merged into...

I'm less concerned about the complexity of the code than about de potential security implications of the PR and potential corner cases not working (fetch). It may be better to...

Thank you for the explanation. I have a similar use-case. I passed the attributes city and street as the group "city/street", without modifying the filename, wich produces a nice tree...

Okay, so after some reflexion, I think this PR should be changed in following points: - a new output, for example CustomFile or DynamicFile, should be created, the File output...

It seems to me that we currently cannot find a common position on this PR. I understand the point of @ytti, but it does not support node variables, and I...