Do you mean you want to include files that are not in the compiled swf link report?
On Aug 26, 10:34 am, Marvin Froeder <[email protected]> wrote: > I was thinking, may be the best approach is scan files, so I can use the > include directive to ensure coverage on this inclusion classes.... > > On Thu, Aug 26, 2010 at 11:32 AM, Daniel Teixeira > <[email protected]>wrote: > > > For the same reason Jesse mentioned above. > > > Quoted from Jesse earlier in this thread: > > > "The key point in the sample (so that you don't have to open it to > > understand > > the problem) is that it contains a class file with an AS3 include > > directive. > > Ie, the structure is as follows: > > > - src/main/flex > > - App.as (uses an "include" to bring in sampleInclude.as) > > - sampleInclude.as > > > With the packaging set to swf, and the sourceFile set to App.as, the > > "compile" operation performs correctly. However, "test" attempts to > > compile > > both App.as and sampleInclude.as separately. This fails, as > > sampleInclude.as is only intended to be included in other files (never > > compiled by itself directly)." > > > On Aug 26, 10:24 am, Marvin Froeder <[email protected]> wrote: > > > Just to remind me, why shouldn't it compile all files? > > > > On Thu, Aug 26, 2010 at 11:06 AM, Daniel Teixeira > > > <[email protected]>wrote: > > > > > Hi Jesse/VELO, > > > > Any updates with the link report scanner? I encountered this same > > > > issue so I decided to write a LinkReportScanner that extends the > > > > existing functionality of the DirectoryScanner and adds the ability to > > > > just include the as/mxml files that are specified in the link report > > > > (as discussed in this thread). It's a simple solution but it works. > > > > I will attach the changes to the issue ticket if you guys haven't > > > > already started working on an "official" solution. > > > > > Thanks, > > > > Daniel > > > > > On Jul 27, 4:11 pm, Marvin Froeder <[email protected]> wrote: > > > > > Sure, I will apply it right away.... I may got to the file thing in > > the > > > > > future, but for sure not gonna happen anytime soon, so it is likely > > your > > > > > change will be there for ever =D > > > > > > On Tue, Jul 27, 2010 at 5:06 PM, Jesse Sightler < > > > > [email protected]>wrote: > > > > > > > I don't really see how it would be slower than reading and parsing > > > > hundreds > > > > > > (if not thousands) of separate files. It also brings in the issue > > of > > > > > > parsing as code to accurately determine if it includes classes. > > > > > > > Honestly, I'd greatly prefer a link-report based solution to > > something > > > > > > based on guessing the files used. I don't really care about > > coverage > > > > for > > > > > > files that aren't in the SWF anyway. :) Really, I could see where > > some > > > > > > people would prefer to have both options available to them. > > > > > > > If I send a prototype of this approach (link-report parsing), would > > it > > > > be > > > > > > likely to get considered for inclusion? > > > > > > > Thanks, > > > > > > Jess > > > > > > > On Tue, Jul 27, 2010 at 12:13 PM, Marvin Froeder < > > [email protected] > > > > >wrote: > > > > > > >> link-report would be more accurate (I guess), but it would be > > slower > > > > > >> (parsing xml) and still my left classes behind..... test code > > coverage > > > > > >> should check coverage of all classes, not only the used by the > > > > APP..... > > > > > > >> This advanced scanner (as a configurable thing) I'm more willing > > to > > > > > >> include on flexmojos. > > > > > > >> VELO > > > > > > >> On Tue, Jul 27, 2010 at 1:00 PM, Jesse Sightler < > > > > [email protected] > > > > > >> > wrote: > > > > > > >>> I think that could work for most cases. I'm wondering, though... > > > > would > > > > > >>> that actually be easier than a link-report scanner? It seems > > like > > > > that > > > > > >>> would be the most accurate way to do it. > > > > > > >>> In any case, I do think that it would work for us. It would > > force us > > > > to > > > > > >>> remove some non-compilable class files, but I don't really see > > that > > > > as a bad > > > > > >>> thing. :) > > > > > > >>> Thanks, > > > > > >>> Jess > > > > > > >>> On Tue, Jul 27, 2010 at 10:45 AM, Marvin Froeder < > > [email protected] > > > > >wrote: > > > > > > >>>> Ithink a better approach would be an advanced class scanner... > > that > > > > > >>>> checks if the .as file is a class, probably just check for class > > > > keyword > > > > > >>>> presence.... > > > > > > >>>> What do you think? > > > > > > >>>> Sent from DROID > > > > > > >>>> Em 26/07/2010 23:09, "Jesse Sightler" <[email protected] > > > > > >>>> >escreveu: > > > > > > >>>> LOL, that is fine. At least the attached patch there gets us > > past > > > > our > > > > > >>>> problem for now. > > > > > > >>>> We look forward to seeing a more official solution as well. :) > > > > > > >>>> Thanks, > > > > > >>>> Jess > > > > > > >>>> On Mon, Jul 26, 2010 at 9:44 PM, Marvin Froeder < > > [email protected]> > > > > > >>>> wrote: > > > > > > >>>>> > You did convinced me this need an different approach, you > > just > > > > didn't > > > > > >>>>> convinced me that the sugg... > > > > > > >>>>> -- > > > > > > >>>>> > You received this message because you are subscribed to the > > > > Google > > > > > >>>>> > Groups "Flex Mojos" group. > > > > > >>>>> > ... > > > > > > >>>> -- > > > > > >>>> You received this message because you are subscribed to the > > Google > > > > > >>>> Groups "Flex Mojos" group. > > > > > >>>> To... > > > > > > >>>> -- > > > > > >>>> You received this message because you are subscribed to the > > Google > > > > > >>>> Groups "Flex Mojos" group. > > > > > >>>> To post to this group, send email to > > [email protected] > > > > > >>>> To unsubscribe from this group, send email to > > > > > >>>> [email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > > >>>> For more options, visit this group at > > > > > >>>>http://groups.google.com/group/flex-mojos > > > > > > >>>>http://flexmojos.sonatype.org/ > > > > > > >>> -- > > > > > >>> You received this message because you are subscribed to the > > Google > > > > > >>> Groups "Flex Mojos" group. > > > > > >>> To post to this group, send email to [email protected] > > > > > >>> To unsubscribe from this group, send email to > > > > > >>> [email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > > >>> For more options, visit this group at > > > > > >>>http://groups.google.com/group/flex-mojos > > > > > > >>>http://flexmojos.sonatype.org/ > > > > > > >> -- > > > > > >> You received this message because you are subscribed to the Google > > > > > >> Groups "Flex Mojos" group. > > > > > >> To post to this group, send email to [email protected] > > > > > >> To unsubscribe from this group, send email to > > > > > >> [email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > > >> For more options, visit this group at > > > > > >>http://groups.google.com/group/flex-mojos > > > > > > >>http://flexmojos.sonatype.org/ > > > > > > > -- > > > > > > You received this message because you are subscribed to the Google > > > > > > Groups "Flex Mojos" group. > > > > > > To post to this group, send email to [email protected] > > > > > > To unsubscribe from this group, send email to > > > > > > [email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > > > For more options, visit this group at > > > > > >http://groups.google.com/group/flex-mojos > > > > > > >http://flexmojos.sonatype.org/ > > > > > -- > > > > You received this message because you are subscribed to the Google > > > > Groups "Flex Mojos" group. > > > > To post to this group, send email to [email protected] > > > > To unsubscribe from this group, send email to > > > > [email protected]<flex-mojos%[email protected]> > > <flex-mojos%[email protected]<flex-mojos%[email protected]> > > > > > For more options, visit this group at > > > >http://groups.google.com/group/flex-mojos > > > > >http://flexmojos.sonatype.org/ > > > -- > > You received this message because you are subscribed to the Google > > Groups "Flex Mojos" group. > > To post to this group, send email to [email protected] > > To unsubscribe from this group, send email to > > [email protected]<flex-mojos%[email protected]> > > For more options, visit this group at > >http://groups.google.com/group/flex-mojos > > >http://flexmojos.sonatype.org/ > > -- You received this message because you are subscribed to the Google Groups "Flex Mojos" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/flex-mojos http://flexmojos.sonatype.org/
