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.

Sign up for GitHub

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

Jump to bottom

Flac file permissions #284

Closed
xmixahlx opened this issue Jul 26, 2018 · 1 comment · Fixed by #407
Closed

Flac file permissions #284

xmixahlx opened this issue Jul 26, 2018 · 1 comment · Fixed by #407
Assignees
@JoeLametta
Labels
Accepted Accepted issue on our roadmap Bug Generic bug: can be used together with more specific labels Priority: medium Medium priority
Milestone
0.8.0

Comments

@xmixahlx
Copy link

whipper writes the document types as 644 but the flac files as 600, which doesn't enable read permissions to your favorite media server app.
@RecursiveForest RecursiveForest self-assigned this Jul 26, 2018
@RecursiveForest RecursiveForest added the Bug Generic bug: can be used together with more specific labels label Jul 26, 2018
@MerlijnWajer
Copy link
Collaborator

I guess we should figure out why the umask is not respected. Probably because of my usage of tempfiles.

@JoeLametta JoeLametta added this to the 2.0 milestone Nov 13, 2018
@JoeLametta JoeLametta added Accepted Accepted issue on our roadmap Priority: medium Medium priority Needed: patch A pull request is required Needed: discussion More discussion needed before anything can be done (or still no agreement has been reached) labels Nov 13, 2018
@tlc tlc mentioned this issue Jan 19, 2019
Open
@JoeLametta JoeLametta modified the milestones: 2.0, 1.0 Jan 20, 2019
@JoeLametta JoeLametta self-assigned this Jan 20, 2019
@JoeLametta JoeLametta changed the title flac file permissions Flac file permissions Jan 20, 2019
JoeLametta added a commit that referenced this issue Jul 7, 2019
@JoeLametta
Set FLAC files permissions to 0644
1c6fe42 
Fixes #284.

Signed-off-by: JoeLametta <[email protected]>
@JoeLametta JoeLametta mentioned this issue Jul 7, 2019
Merged
@JoeLametta JoeLametta added the Status: in progress Issue/pull request which is currently being worked on label Jul 7, 2019
@JoeLametta JoeLametta modified the milestones: 1.0, 0.8.0 Oct 21, 2019
@JoeLametta JoeLametta removed Status: in progress Issue/pull request which is currently being worked on Needed: patch A pull request is required Needed: discussion More discussion needed before anything can be done (or still no agreement has been reached) labels Oct 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JoeLametta JoeLametta

Labels
Accepted Accepted issue on our roadmap Bug Generic bug: can be used together with more specific labels Priority: medium Medium priority
Projects
None yet
Milestone
0.8.0
Development

Successfully merging a pull request may close this issue.

Set FLAC files permissions to 0644 whipper-team/whipper
4 participants
@RecursiveForest @MerlijnWajer @JoeLametta @xmixahlx