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

#125: Preserve the original file permission by default #126

Closed
wants to merge 1 commit into from

Conversation

xerial
Copy link

@xerial xerial commented Jan 23, 2018

No description provided.

Copy link
Member

@eed3si9n eed3si9n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On hold until we figure out what's going on here.
#125
Likely recommendation is to just use setPermissions method introduced in #76 if that's what's needed.

@typesafe-tools
Copy link

The validator has checked the following projects against Scala 2.12,
tested using dbuild, projects built on top of each other.

Project Reference Commit
sbt 1.x sbt/sbt@8cfec5b
zinc 1.x sbt/zinc@2fade53
io pull/126/head ddd1c1e
librarymanagement 1.x sbt/librarymanagement@79c7483
util 1.x sbt/util@2022713
website 1.x

❌ The result is: FAILED
(restart)

dwijnand
dwijnand previously approved these changes Jan 24, 2018
@@ -16,4 +16,6 @@ type CopyOptions {
## If `true` the executable properties are copied.
preserveExecutable: boolean! = true @since("0.0.1")

## If `true` the file permissions are copied. Setting this `true` disregards the effect of preserveExcecutable flag
preservePermission: boolean! = true @since("0.0.2")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be since 1.1.3 or 1.2.0 (depending where we merge it) btw

@dwijnand dwijnand dismissed their stale review January 24, 2018 09:50

misclicked - I meant just to leave a comment

@eed3si9n
Copy link
Member

eed3si9n commented Feb 8, 2018

Closing this.

@eed3si9n eed3si9n closed this Feb 8, 2018
@eed3si9n eed3si9n removed the ready label Feb 8, 2018
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

Successfully merging this pull request may close these issues.

4 participants