-
Notifications
You must be signed in to change notification settings - Fork 41
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
Tests fail with File::ShareDir::ProjectDistDir 0.5.2 #35
Comments
Placing a .devdir file in t/ directory seems to work quite nicely |
Hey, Thanks for pointing this out! I'm in the middle of a big move, so will take a look as soon as I can get to it. I am honestly a bit unhappy with this change in ProjectDistDir... I'll have to try and understand that. Thanks! John From: patmooney notifications@github.com Tests run with this error: Can't find a share () for Local |
Had the same issue, the .devdir File in t/ works |
@vkroll , @patmooney hey so I finally have some time clear to look at this, but sadly I can't replicate the issue. I've installed the newest ::ProjectDistDir and all. Can one of you add in comment the complete run and fail, and the steps you take to generate it, that way I can track it down. Thanks --John |
A rather drawn out method: Use of uninitialized value $sharedir in concatenation (.) or string at /home/patrick/.cpanm/work/1381256651.7654/DBIx-Class-Migration-0.041/blib/lib/DBIx/Class/Migration/ShareDirBuilder.pm line 33. |
I the specific issue is with Path::IsDev and should have made that clear so I apologise! |
I thought I did a totally clean local lib and should have gotten the new Path::IsDev Ok I will try again! thanks!!!! -john On Tuesday, October 8, 2013 1:32 PM, patmooney notifications@github.com wrote: I the specific issue is with Path::IsDev and should have made that clear so I apologise! |
Well, I can't replicate this issue no matter what, but I'll just go ahead and do a release with that t/.devdir and hope it just works. Let me know! --John |
I believe Dist::Zilla::Plugin::PruneCruft is removing t/.devdir in your new release |
Tests run with this error: Can't find a share () for Local
The new version of ProjectDistDir no longer looks for a lib/ directory to dictate the project root/ so the share/ directory is not being correctly located in t/
File::ShareDir::ProjectDistDir 0.5.2 searches upwards of the given module and looks for any of the following
META.json
META.yml
Change(s|log)(|[.][^.\s]+)
GNUmakefile
makefile
Makefile
.devdir
t/
xt/
MYMETA.json
MYMETA.yml
dist.ini
Makefile.PL
Build.PL
The text was updated successfully, but these errors were encountered: