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

-u option does not work #6

Closed
vizmi opened this issue Apr 22, 2015 · 6 comments
Closed

-u option does not work #6

vizmi opened this issue Apr 22, 2015 · 6 comments

Comments

@vizmi
Copy link
Contributor

vizmi commented Apr 22, 2015

Hi Calvin,

apologies for the confusion: I was somewhat convinced the isFile() filtering will fix the -u issue too. It turned out it is not. The 4 test cases with -u are still failing for me.
I have committed the a version to my clone that I believe fixes the issue (the path.normalize call we've been going back and forth) and some logging that helps further diagnostics in the future. I hope you will find it useful.
Let me know if you need a pull request from my end.
@davidfurlong
Copy link

Still broken :S

@vizmi
Copy link
Contributor Author

vizmi commented Aug 4, 2018

My changes never made it to the main code line

@calvinmetcalf
Copy link
Owner

sorry I never responded, could you open a pull ?

@HoldYourWaffle
Copy link

Is there any update on this?

@vizmi
Copy link
Contributor Author

vizmi commented Feb 23, 2019

My changes were lost long before this conversation reopened, sorry.
I am affraid only the original author can help here.

@vizmi vizmi closed this as completed Jan 13, 2022
@vizmi
Copy link
Contributor Author

vizmi commented Jan 13, 2022

too old to fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@davidfurlong @vizmi @calvinmetcalf @HoldYourWaffle