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

Fix chown invocation #228

Merged
merged 1 commit into from Jul 9, 2018
Merged

Fix chown invocation #228

merged 1 commit into from Jul 9, 2018

Conversation

tomdee
Copy link
Contributor

@tomdee tomdee commented Jul 9, 2018

  • I have added or updated the specs.
  • I have verified that the specs pass on my computer.
  • I have not attempted to bump, or alter versions.
  • This is a documentation change.
  • This is a source change.

Fix issue introduced by e96998e

Hit the following error when running the latest jekyll/jekyll:pages
image

docker run --rm -ti -e JEKYLL_UID=`id -u` jekyll/jekyll:pages jekyll build
BusyBox v1.27.2 (2018-06-06 09:08:44 UTC) multi-call binary.

Usage: chown [-RhLHPcvf]... USER[:[GRP]] FILE...

Change the owner and/or group of each FILE to USER and/or GRP

	-R	Recurse
	-h	Affect symlinks instead of symlink targets
	-L	Traverse all symlinks to directories
	-H	Traverse symlinks on command line only
	-P	Don't traverse symlinks (default)
	-c	List changed files
	-v	List all files
	-f	Hide errors

@tomdee tomdee requested a review from envygeeks as a code owner July 9, 2018 21:43
@tomdee
Fix chown invocation
bc385da 
Fix issue introduced by e96998e

Hit the following error when running the latest jekyll/jekyll:pages
image

```
docker run --rm -ti -e JEKYLL_UID=`id -u` jekyll/jekyll:pages jekyll build
BusyBox v1.27.2 (2018-06-06 09:08:44 UTC) multi-call binary.

Usage: chown [-RhLHPcvf]... USER[:[GRP]] FILE...

Change the owner and/or group of each FILE to USER and/or GRP

	-R	Recurse
	-h	Affect symlinks instead of symlink targets
	-L	Traverse all symlinks to directories
	-H	Traverse symlinks on command line only
	-P	Don't traverse symlinks (default)
	-c	List changed files
	-v	List all files
	-f	Hide errors
```
envygeeks
envygeeks approved these changes Jul 9, 2018
View reviewed changes
Copy link
Owner

@envygeeks envygeeks left a comment

Choose a reason for hiding this comment

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

This MacBooks keyboard. Sorry about that!

@envygeeks envygeeks merged commit b7afd94 into envygeeks:master Jul 9, 2018
@envygeeks
Copy link
Owner

Thanks 💖

@tomdee tomdee deleted the fix-pages branch July 9, 2018 22:19
tomdee added a commit to tomdee/docker-1 that referenced this pull request Jul 9, 2018
@tomdee
Add user:group to entrypoint script
1276b59 
Follow on from envygeeks#228

Turns out it wasn't just the -R from envygeeks#228 that needed to be changed, the
user:group was missing too
@tomdee tomdee mentioned this pull request Jul 9, 2018
Merged
5 tasks
envygeeks pushed a commit that referenced this pull request Jul 9, 2018
@tomdee @envygeeks
Add user:group to entrypoint script (#229)
7f2f206 
Follow on from #228

Turns out it wasn't just the -R from #228 that needed to be changed, the
user:group was missing too
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@envygeeks envygeeks envygeeks approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tomdee @envygeeks