Page MenuHomeFreeBSD

Kill EXTRACT_PRESERVE_OWNERSHIP, update some comments
ClosedPublic

Authored by antoine on Nov 18 2014, 11:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sep 11 2024, 11:40 AM
Unknown Object (File)
Sep 7 2024, 4:58 PM
Unknown Object (File)
Sep 5 2024, 10:38 PM
Unknown Object (File)
Sep 5 2024, 10:38 PM
Unknown Object (File)
Aug 25 2024, 12:36 AM
Unknown Object (File)
Jul 20 2024, 4:39 PM
Unknown Object (File)
Jun 29 2024, 12:43 PM
Unknown Object (File)
Jun 17 2024, 3:59 PM
Subscribers

Details

Reviewers
antoine
Group Reviewers
portmgr
Summary

Kill EXTRACT_PRESERVE_OWNERSHIP, update some comments

Test Plan

Verify it still builds

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

antoine retitled this revision from to Kill EXTRACT_PRESERVE_OWNERSHIP, update some comments.
antoine updated this object.
antoine edited the test plan for this revision. (Show Details)
antoine added a reviewer: antoine.

I accept my revision

This revision is now accepted and ready to land.Dec 14 2014, 11:05 AM

What is the "Why" here? This is useful for NFS checkouts where using root squash.

There was no mention of root-squash in the commit log or in the comment so I didn't think about it.

I can put back the .if !defined(EXTRACT_PRESERVE_OWNERSHIP) around the CHMOD/CHOWN.

For --no-same-owner --no-same-permissions, I think it doesn't harm with root-squash. Do you confirm?

In D1189#10, @antoine wrote:

There was no mention of root-squash in the commit log or in the comment so I didn't think about it.

I can put back the .if !defined(EXTRACT_PRESERVE_OWNERSHIP) around the CHMOD/CHOWN.

For --no-same-owner --no-same-permissions, I think it doesn't harm with root-squash. Do you confirm?

I'll experiment and get back to you.