Summary: | [PATCH] More advanced suidctl Portage feature | ||
---|---|---|---|
Product: | Portage Development | Reporter: | email_deleted_GqKU |
Component: | Enhancement/Feature Requests | Assignee: | Portage team <dev-portage> |
Status: | RESOLVED DUPLICATE | ||
Severity: | enhancement | ||
Priority: | High | ||
Version: | 2.1 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 193766 | ||
Attachments: |
Patch for an advanced version of the suidctl Portage feature
Default configuration file (with comments) for the feature |
Description
email_deleted_GqKU
2006-10-17 20:52:57 UTC
Created attachment 99915 [details, diff]
Patch for an advanced version of the suidctl Portage feature
Note that the modifications have been only lightly tested... and you might want to shorten some variable names, if you do not like long names :) (and maybe reformat some other things, if you do not like this style and/or prefer to be consistent -which I understand).
As said in the original comment, this change, as implemented by this patch, is not compatible with current implementation of the suidctl features (though a new name has been used for the feature, so a simple warning might do -well, you might also add some compatibility).
Created attachment 99916 [details]
Default configuration file (with comments) for the feature
As said, the file (which does not have the same name as the previous one), does not have the same format.
The format being more complex, the default configuration file should most probably be included by default, either as "/etc/portage/setid-control.conf", or as something like "/usr/share/portage/setid-control.conf.default" (which is used in the posted patch, and copied to "/etc/portage/setid-control.conf", if the file is not present).
I'd more inclined to outsource this features as a bashrc hook. suidctl doesn't seem widely used, so I'm not much for adding extensive patches to it (even if it's mostly comments). Solar: IIRC you added suidctl originally (or was that sfperms?), so what do you think about this? (In reply to comment #3) > I'd more inclined to outsource this features as a bashrc hook. suidctl doesn't > seem widely used, so I'm not much for adding extensive patches to it (even if > it's mostly comments). > > Solar: IIRC you added suidctl originally (or was that sfperms?), so what do you think about this? both.. but to be honest I'm in favor of dropping suidctl all together. Nick merged it not the way I had quite intended. *** This bug has been marked as a duplicate of bug 396153 *** |