Re: Mason tidy

On Sep 14, 2012, at 1:41 PM, Shane McCarron wrote:

> Also it looks like there is a Test::Class dependency that was undeclared.
> 

2.5.1 depends on Test::Class::Most, that should do it? But I've added missing dependencies Capture::Tiny, File::Temp, and IPC::System::Simple. Just realized that I used a broken regex when searching for modules I used. :)

> I noticed a couple if things when I got it installed and tried to run it:
> I have a (bad) habit of having %perl blocks that are not freestanding.  In otherwords, there is a perl section that has a condition or the start of a block in it, then there is some HTML outside of a perl block (possibly with inline Mason), then there is another perl section (or an inline Mason) that closes the block.  I understand that this is not supported, but I am not sure how to tell Mason::Tidy to permit this (bad) habit.
Ahh, I see, so you'd want the <%perl> sections to combine together the way that %-lines currently do. That's actually quite reasonable, but it wlll take some more work!
> When running masontidy on a sample file I get a lot of output like "Ignoring -b; you may not specify a destination stream and -b together".  Not sure what that means, but I assume it is something from perltidy.
No idea, did you put -b in your perltidy options list? You shouldn't need to.
> When building on a Windows machine the generated .bat file works fine, but the generated native perl has the wrong #! line in it.  Not sure if there is a way to fix that, but if there is then it would work right when called from a Cygwin shell on Windows (I use that for a development environment sometimes).
I just have #!/usr/bin/perl at the top of bin/mason, I've always thought that was the correct thing to put. I don't have a Windows environment but let me know if you figure out a way to fix it.

> Thanks for all your hard work!

Yup, thanks for the feedback!

Jon

------------------------------------------------------------------------------
Got visibility?
Most devs has no idea what their production app looks like.
Find out how fast your code is with AppDynamics Lite.
http://ad.doubleclick.net/clk;262219671;13503038;y?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html

_______________________________________________
Mason-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mason-users