"touch -c" now runs in slices of 100 (instead of 1000) #49

Merged
merged 1 commit into from Jan 11, 2013

Conversation

Projects
None yet
4 participants
@sibblingz

We got a "Argument list too long" error during deployment, because our deploy-to path is quite lengthy. Ideally, this list should be sized by character count and not file count, but this will do for now.

@tuttinator

This comment has been minimized.

Show comment
Hide comment
@tuttinator

tuttinator Jan 11, 2013

+1 Awesome - thanks for that. I just ran into that issue now.

+1 Awesome - thanks for that. I just ran into that issue now.

ndbroadbent added a commit that referenced this pull request Jan 11, 2013

Merge pull request #49 from sibblingz/master
"touch -c" now runs in slices of 100 (instead of 1000)

@ndbroadbent ndbroadbent merged commit ad9b681 into ndbroadbent:master Jan 11, 2013

1 check passed

default The Travis build passed
Details
@ndbroadbent

This comment has been minimized.

Show comment
Hide comment
@ndbroadbent

ndbroadbent Jan 11, 2013

Owner

Great, thanks! Have released this change in 0.3.6

Owner

ndbroadbent commented Jan 11, 2013

Great, thanks! Have released this change in 0.3.6

@tuttinator

This comment has been minimized.

Show comment
Hide comment
@tuttinator

tuttinator Jan 11, 2013

Wow - thanks, amazingly fast response.

Wow - thanks, amazingly fast response.

@sibblingz

This comment has been minimized.

Show comment
Hide comment
@sibblingz

sibblingz Jan 11, 2013

+1 that, very quick! I can get rid of my fork now.

+1 that, very quick! I can get rid of my fork now.

@ahoward

This comment has been minimized.

Show comment
Hide comment
@ahoward

ahoward Jan 13, 2013

yuk. why not simply

  FileUtils.touch(paths.select{|path| test(?e, path)})

yuk. why not simply

  FileUtils.touch(paths.select{|path| test(?e, path)})
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment