Skip to content

fapolicyd-cli: Add option to print the file name#293

Open
berndfinger wants to merge 2 commits intolinux-application-whitelisting:mainfrom
berndfinger:issue-292
Open

fapolicyd-cli: Add option to print the file name#293
berndfinger wants to merge 2 commits intolinux-application-whitelisting:mainfrom
berndfinger:issue-292

Conversation

@berndfinger
Copy link

... along with the file type.

Solves issue #292.

... along with the file type.

Solves issue linux-application-whitelisting#292.

Signed-off-by: Bernd Finger <bfinger@redhat.com>
@radosroka
Copy link
Member

@berndfinger, please rename --ftype_ext at least to --ftype-ext so the options are consistent. I personally prefer something different like --ftype --print-file-name

What do you think?

Relates to linux-application-whitelisting#292.

Signed-off-by: Bernd Finger <bfinger@redhat.com>
@berndfinger
Copy link
Author

@berndfinger, please rename --ftype_ext at least to --ftype-ext so the options are consistent. I personally prefer something different like --ftype --print-file-name

What do you think?

Both suggestions make sense to me. For the time being, I replaced the underscore by dash, so the options are now consistent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants