__acl: remove deprecated parameters #43
Labels
No labels
bugfix
cleanup
discussion
documentation
doing
done
feature
improvement
packaging
Stale
testing
TODO
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
ungleich-public/cdist#43
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ander Can you remove
__acldeprecated parameters and prepare MR?In the ++minor we will remove them.
mentioned in commit
f96f23e970closed via merge request !933
mentioned in merge request !933
sigh, time flies. sorry for taking this long, I have no excuse but laziness 👹
but take a look at !933
@ander Is it feasible that you do this in a week or two?
This question came up when I suggested "cdist-museum" for old and deleted types in cdist core. @poljakowski raised the question if this should also apply to the deprecated parameters in
__acl.IMO: If no use case breaks and it's just a matter of switching from one parameter to another, there is no reason to archive the old version.
currently backwards compatibility does exactly that - maps deprecated parameters to
--entryand also emits warning.I think we have waited long enough.
I'll deal with this after dance around
__downloadand__unpackends in one way or another.I'm continuing the discussion of !899 here, because it doesn't really fit there.
I don't know POSIX ACLs well, but I'm wondering if all previous use cases of the deprecated parameters could be mapped to a new
--entryparameter?If not, which cases do not work anymore if the deprecated parameters are removed?
mentioned in merge request !899