-
Notifications
You must be signed in to change notification settings - Fork 18
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
Wrong output directory created when using current directory pattern (./) in src #29
Comments
jeffreybarke
added a commit
to jeffreybarke/grunt-smushit
that referenced
this issue
Mar 30, 2014
* Update getSubPath function in /tasks/smushit.js to normalize filepath parameter similar to how the fileDir parameter is normalized. This fixes heldr#29, in which src patterns such as ./img create an extraneous g/ directory in the dest directory. * Update version in package.json to v1.2.1. * Update release history in README with v.1.2.1 deets.
Merged
heldr
added a commit
that referenced
this issue
Apr 2, 2014
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When using
src
patterns such as./img
instead ofimg
, an extraneousg/
directory is created in thedest
dir. I don't see why onesrc
pattern is to be preferred over the other; both are valid paths and both should work.This is essentially a duplicate of #28 that @dmcass opened. There is substance here, however his fix was incorrect. Instead of hardcoding a
+ 1
(cc764d9), it seems better to alwayspath.normalize
filepath
in the same way thatfileDir
is normalized.The text was updated successfully, but these errors were encountered: