Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unimplemented keywords: 'flags' and 'nlink' (for tar archives) #40

Open
imsteev opened this issue Jul 25, 2016 · 10 comments
Open

unimplemented keywords: 'flags' and 'nlink' (for tar archives) #40

imsteev opened this issue Jul 25, 2016 · 10 comments

Comments

@imsteev
Copy link

imsteev commented Jul 25, 2016

[schung@localhost Desktop] $ sudo `which gomtree` -f mount.mtree -p /mnt
2016/07/25 19:43:01 Unknown keyword "flags" for file "."
@imsteev imsteev closed this as completed Jul 26, 2016
@vbatts
Copy link
Owner

vbatts commented Jul 26, 2016

why closed?

On Mon, Jul 25, 2016 at 10:27 PM, Stephen Chung notifications@github.com
wrote:

Closed #40 #40.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#40 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEF6bRDiaBb1RoU1JpX4Eum7E35lzFkks5qZXB5gaJpZM4JUp6x
.

@imsteev
Copy link
Author

imsteev commented Jul 26, 2016

I realized there's a lot of other keywords not implemented, so I thought it'd make more sense to discuss which unimplemented keywords need attention, rather than just pointing out one.

@imsteev
Copy link
Author

imsteev commented Jul 26, 2016

I could re-open and edit this issue accordingly if you want

@imsteev
Copy link
Author

imsteev commented Jul 26, 2016

With regard to "important" keywords, and to align with upstream mtree, 'flags' and 'nlink' (tar archive only) are keywords that should be looked at next.

@imsteev imsteev reopened this Jul 26, 2016
@imsteev imsteev changed the title implement 'flags' keyword unimplemented keywords: 'flags' and 'nlink' (for tar archives) Jul 26, 2016
@vbatts
Copy link
Owner

vbatts commented Jul 26, 2016

sure thing. It's a good thing to implement these keywords. Though having them be included in the default set may not be needed. Especially ones like nlink, as technology like ostree use hardlinks heavily, so the nlink on disk will almost always be different on anything given host.

@vbatts
Copy link
Owner

vbatts commented Jul 26, 2016

nlink is already implemented

@imsteev
Copy link
Author

imsteev commented Jul 26, 2016

yeah but for tar archives we would have to manually keep track of links

@vbatts
Copy link
Owner

vbatts commented Jul 26, 2016

oh i see. yeah, keeping track of hardlinks in tar archives is more work

@imsteev
Copy link
Author

imsteev commented Aug 12, 2016

nlink keyword implemented for tar archives, although nlink for directory Entry's have not been implemented.

@vbatts
Copy link
Owner

vbatts commented Oct 31, 2016

I've just stubbed flags, but we can hold on these still.

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

No branches or pull requests

2 participants