You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flex.apache.org by Josh Tynjala <jo...@gmail.com> on 2015/07/11 01:06:49 UTC
[FalconJX jsc] Compiler error when using classes in packages.
It looks like the jsc output type is having trouble with classes when
they're in packages. See the error below:
Could not find file for class: feathers.controls.SimpleButton
java.lang.RuntimeException: Unable to find JavaScript filePath for class:
newPack.OtherClass
at
org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWriter.java:174)
at
org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWriter.java:192)
at
org.apache.flex.compiler.internal.graph.GoogDepsWriter.buildDB(GoogDepsWriter.java:127)
at
org.apache.flex.compiler.internal.graph.GoogDepsWriter.getListOfFiles(GoogDepsWriter.java:79)
at
org.apache.flex.compiler.internal.codegen.mxml.flexjs.MXMLFlexJSPublisher.publish(MXMLFlexJSPublisher.java:325)
at org.apache.flex.compiler.clients.MXMLJSC.compile(MXMLJSC.java:467)
at
org.apache.flex.compiler.clients.MXMLJSC._mainNoExit(MXMLJSC.java:346)
at org.apache.flex.compiler.clients.MXMLJSC.mainNoExit(MXMLJSC.java:272)
at
org.apache.flex.compiler.clients.MXMLJSC.staticMainNoExit(MXMLJSC.java:231)
at org.apache.flex.compiler.clients.MXMLJSC.main(MXMLJSC.java:177)
File not found: feathers.controls.SimpleButton
If I put the SimpleButton class into the top-level package, it compiles
without error. Any ideas?
- Josh
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Josh Tynjala <jo...@gmail.com>.
Alex,
I just wanted to let you know that I tried these changes today, and
everything seems to be working nicely.
- Josh
On Wed, Jul 15, 2015 at 2:07 PM, Alex Harui <ah...@adobe.com> wrote:
>
>
> On 7/15/15, 3:50 AM, "Erik de Bruin" <er...@ixsoftware.nl> wrote:
>
> >>
> >> Hmm. I think I’ll stop what I’m doing (which is trying to make UI
> >>widgets
> >> handle more CSS) and make the switch back to “.” I didn’t realize it
> >> would get messy with the Language class.
> >>
> >
> >Yes, please.
>
> OK, just pushed it. I still haven’t heard back from MS about MSDN so the
> CI builds are still down. Let me know if you run into issues.
>
> The number of warnings is now reasonable low. I am going to make one more
> change to get rid of a frequent warning, then go back to my other stuff.
>
> I think we have two open issues for folks to work on or at least offer
> opinions:
>
> 1) how to save goog.bind results so addEventListener and
> removeEventListener work in direct-to-HTMLElement apps?
> 2) how to output new globals in strict mode.
>
> Thanks,
> -Alex
>
>
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Alex Harui <ah...@adobe.com>.
On 7/15/15, 3:50 AM, "Erik de Bruin" <er...@ixsoftware.nl> wrote:
>>
>> Hmm. I think I’ll stop what I’m doing (which is trying to make UI
>>widgets
>> handle more CSS) and make the switch back to “.” I didn’t realize it
>> would get messy with the Language class.
>>
>
>Yes, please.
OK, just pushed it. I still haven’t heard back from MS about MSDN so the
CI builds are still down. Let me know if you run into issues.
The number of warnings is now reasonable low. I am going to make one more
change to get rid of a frequent warning, then go back to my other stuff.
I think we have two open issues for folks to work on or at least offer
opinions:
1) how to save goog.bind results so addEventListener and
removeEventListener work in direct-to-HTMLElement apps?
2) how to output new globals in strict mode.
Thanks,
-Alex
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Erik de Bruin <er...@ixsoftware.nl>.
>
> Hmm. I think I’ll stop what I’m doing (which is trying to make UI widgets
> handle more CSS) and make the switch back to “.” I didn’t realize it
> would get messy with the Language class.
>
Yes, please.
EdB
--
Ix Multimedia Software
Jan Luykenstraat 27
3521 VB Utrecht
T. 06-51952295
I. www.ixsoftware.nl
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Alex Harui <ah...@adobe.com>.
Hmm. I think I’ll stop what I’m doing (which is trying to make UI widgets
handle more CSS) and make the switch back to “.” I didn’t realize it
would get messy with the Language class.
-Alex
On 7/14/15, 5:18 PM, "Josh Tynjala" <jo...@gmail.com> wrote:
>I don't mind having some local changes on my machine for a little while.
>However, if you think it's a good idea for me to commit them, I will.
>
>I also ran into an issue where GoogDepsWriter was throwing an error about
>createjs.Container being missing, and I had add some code to check if this
>class was defined in any of the SWCs in the external-library-path. What's
>strange is that it wasn't happening every time I compiled. It was
>intermittent. I don't know what was going on there, but with this code
>that
>I added, it seems to have helped.
>
>- Josh
>
>On Tue, Jul 14, 2015 at 2:04 PM, Michael Schmalle
><teotigraphixllc@gmail.com
>> wrote:
>
>> Josh,
>>
>> Would it be easier for you to just put what you have changed in if()
>> statements and test whether the output type is JSC, then commit your
>> changes?
>>
>> It's kind of hackish but then you would be back in sync. I really don't
>>see
>> a problem with the right now until Alex is ready to change things.
>>
>> Mike
>>
>> On Tue, Jul 14, 2015 at 4:38 PM, Josh Tynjala <jo...@gmail.com>
>> wrote:
>>
>> > If anyone else is interested in using JSC, here are the changes that I
>> > made.
>> >
>> > In the getFilePath() function of GoogDepsWriter, on line 377:
>> >
>> > String classPath = className.replace(".", File.separator);
>> > if(classPath.equals(className))
>> > {
>> > classPath = className.replace("_", File.separator);
>> > }
>> >
>> > As a fallback, I kept the case where _ is replaced because the
>> > org_apache_flex_utils_Language is still named with underscores right
>>now,
>> > and I was getting an error because of it.
>> >
>> > Later in the same function:
>> >
>> > String fwdClassPath = className.replace(".", "/");
>> > if(fwdClassPath.equals(className))
>> > {
>> > fwdClassPath = className.replace("_", "/");
>> > }
>> > String bckClassPath = className.replace(".", "\\");
>> > if(bckClassPath.equals(className))
>> > {
>> > bckClassPath = className.replace("_", "\\");
>> > }
>> >
>> > Finally, this change should also happen in the formatQualifiedName()
>> > function in the JSFlexJSDocEmitter class. The doc comments also
>>currently
>> > reference class names with underscores.
>> >
>> > protected String formatQualifiedName(String name)
>> > {
>> > if (name.contains("goog.") || name.startsWith("Vector."))
>> > return name;
>> > name = name.replaceAll("\\.", ".");
>> > return name;
>> > }
>> >
>> > - Josh
>> >
>> > On Sat, Jul 11, 2015 at 10:22 AM, Josh Tynjala <jo...@gmail.com>
>> > wrote:
>> >
>> > > No rush! I can make the change locally.
>> > >
>> > > - Josh
>> > > On Jul 10, 2015 9:26 PM, "Alex Harui" <ah...@adobe.com> wrote:
>> > >
>> > >> Sorry for the late reply. I had internet issues all day.
>> > >>
>> > >> The problem is that GoogDepsWriter.java is still assuming “_” for
>> > package
>> > >> separators. See line 377.
>> > >> String classPath = className.replace("_", File.separator);
>> > >>
>> > >> I wasn’t planning on switching back to “.” until after next week or
>> so,
>> > >> but I suppose I could shift priorities. Or if you have built your
>>own
>> > >> compiler you can limp along by making this change in your local
>>copy.
>> > >>
>> > >> -Alex
>> > >>
>> > >> On 7/10/15, 4:15 PM, "Josh Tynjala" <jo...@gmail.com> wrote:
>> > >>
>> > >> >Just in case someone gets confused, this line should not include
>> > >> >newPack.OtherClass:
>> > >> >
>> > >> >java.lang.RuntimeException: Unable to find JavaScript filePath for
>> > class:
>> > >> >newPack.OtherClass
>> > >> >
>> > >> >Bad copy-paste job. The real output was
>> feathers.controls.SimpleButton
>> > >> >too.
>> > >> >
>> > >> >On Fri, Jul 10, 2015 at 4:06 PM, Josh Tynjala
>><joshtynjala@gmail.com
>> >
>> > >> >wrote:
>> > >> >
>> > >> >> It looks like the jsc output type is having trouble with classes
>> when
>> > >> >> they're in packages. See the error below:
>> > >> >>
>> > >> >> Could not find file for class: feathers.controls.SimpleButton
>> > >> >> java.lang.RuntimeException: Unable to find JavaScript filePath
>>for
>> > >> >>class:
>> > >> >> newPack.OtherClass
>> > >> >> at
>> > >> >>
>> > >>
>> > >>
>> >
>>
>>>>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDeps
>>>>Wr
>> > >> >>iter.java:174)
>> > >> >> at
>> > >> >>
>> > >>
>> > >>
>> >
>>
>>>>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDeps
>>>>Wr
>> > >> >>iter.java:192)
>> > >> >> at
>> > >> >>
>> > >>
>> > >>
>> >
>>
>>>>org.apache.flex.compiler.internal.graph.GoogDepsWriter.buildDB(GoogDeps
>>>>Wr
>> > >> >>iter.java:127)
>> > >> >> at
>> > >> >>
>> > >>
>> > >>
>> >
>>
>>>>org.apache.flex.compiler.internal.graph.GoogDepsWriter.getListOfFiles(G
>>>>oo
>> > >> >>gDepsWriter.java:79)
>> > >> >> at
>> > >> >>
>> > >>
>> > >>
>> >
>>
>>>>org.apache.flex.compiler.internal.codegen.mxml.flexjs.MXMLFlexJSPublish
>>>>er
>> > >> >>.publish(MXMLFlexJSPublisher.java:325)
>> > >> >> at
>> > >>
>>>>org.apache.flex.compiler.clients.MXMLJSC.compile(MXMLJSC.java:467)
>> > >> >> at
>> > >> >>
>> > org.apache.flex.compiler.clients.MXMLJSC._mainNoExit(MXMLJSC.java:346)
>> > >> >> at
>> > >> >>
>> org.apache.flex.compiler.clients.MXMLJSC.mainNoExit(MXMLJSC.java:272)
>> > >> >> at
>> > >> >>
>> > >>
>> > >>
>> >
>>
>>>>org.apache.flex.compiler.clients.MXMLJSC.staticMainNoExit(MXMLJSC.java:
>>>>23
>> > >> >>1)
>> > >> >> at
>> > org.apache.flex.compiler.clients.MXMLJSC.main(MXMLJSC.java:177)
>> > >> >>
>> > >> >>
>> > >> >> File not found: feathers.controls.SimpleButton
>> > >> >>
>> > >> >> If I put the SimpleButton class into the top-level package, it
>> > compiles
>> > >> >> without error. Any ideas?
>> > >> >>
>> > >> >> - Josh
>> > >> >>
>> > >>
>> > >>
>> >
>>
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Josh Tynjala <jo...@gmail.com>.
I don't mind having some local changes on my machine for a little while.
However, if you think it's a good idea for me to commit them, I will.
I also ran into an issue where GoogDepsWriter was throwing an error about
createjs.Container being missing, and I had add some code to check if this
class was defined in any of the SWCs in the external-library-path. What's
strange is that it wasn't happening every time I compiled. It was
intermittent. I don't know what was going on there, but with this code that
I added, it seems to have helped.
- Josh
On Tue, Jul 14, 2015 at 2:04 PM, Michael Schmalle <teotigraphixllc@gmail.com
> wrote:
> Josh,
>
> Would it be easier for you to just put what you have changed in if()
> statements and test whether the output type is JSC, then commit your
> changes?
>
> It's kind of hackish but then you would be back in sync. I really don't see
> a problem with the right now until Alex is ready to change things.
>
> Mike
>
> On Tue, Jul 14, 2015 at 4:38 PM, Josh Tynjala <jo...@gmail.com>
> wrote:
>
> > If anyone else is interested in using JSC, here are the changes that I
> > made.
> >
> > In the getFilePath() function of GoogDepsWriter, on line 377:
> >
> > String classPath = className.replace(".", File.separator);
> > if(classPath.equals(className))
> > {
> > classPath = className.replace("_", File.separator);
> > }
> >
> > As a fallback, I kept the case where _ is replaced because the
> > org_apache_flex_utils_Language is still named with underscores right now,
> > and I was getting an error because of it.
> >
> > Later in the same function:
> >
> > String fwdClassPath = className.replace(".", "/");
> > if(fwdClassPath.equals(className))
> > {
> > fwdClassPath = className.replace("_", "/");
> > }
> > String bckClassPath = className.replace(".", "\\");
> > if(bckClassPath.equals(className))
> > {
> > bckClassPath = className.replace("_", "\\");
> > }
> >
> > Finally, this change should also happen in the formatQualifiedName()
> > function in the JSFlexJSDocEmitter class. The doc comments also currently
> > reference class names with underscores.
> >
> > protected String formatQualifiedName(String name)
> > {
> > if (name.contains("goog.") || name.startsWith("Vector."))
> > return name;
> > name = name.replaceAll("\\.", ".");
> > return name;
> > }
> >
> > - Josh
> >
> > On Sat, Jul 11, 2015 at 10:22 AM, Josh Tynjala <jo...@gmail.com>
> > wrote:
> >
> > > No rush! I can make the change locally.
> > >
> > > - Josh
> > > On Jul 10, 2015 9:26 PM, "Alex Harui" <ah...@adobe.com> wrote:
> > >
> > >> Sorry for the late reply. I had internet issues all day.
> > >>
> > >> The problem is that GoogDepsWriter.java is still assuming “_” for
> > package
> > >> separators. See line 377.
> > >> String classPath = className.replace("_", File.separator);
> > >>
> > >> I wasn’t planning on switching back to “.” until after next week or
> so,
> > >> but I suppose I could shift priorities. Or if you have built your own
> > >> compiler you can limp along by making this change in your local copy.
> > >>
> > >> -Alex
> > >>
> > >> On 7/10/15, 4:15 PM, "Josh Tynjala" <jo...@gmail.com> wrote:
> > >>
> > >> >Just in case someone gets confused, this line should not include
> > >> >newPack.OtherClass:
> > >> >
> > >> >java.lang.RuntimeException: Unable to find JavaScript filePath for
> > class:
> > >> >newPack.OtherClass
> > >> >
> > >> >Bad copy-paste job. The real output was
> feathers.controls.SimpleButton
> > >> >too.
> > >> >
> > >> >On Fri, Jul 10, 2015 at 4:06 PM, Josh Tynjala <joshtynjala@gmail.com
> >
> > >> >wrote:
> > >> >
> > >> >> It looks like the jsc output type is having trouble with classes
> when
> > >> >> they're in packages. See the error below:
> > >> >>
> > >> >> Could not find file for class: feathers.controls.SimpleButton
> > >> >> java.lang.RuntimeException: Unable to find JavaScript filePath for
> > >> >>class:
> > >> >> newPack.OtherClass
> > >> >> at
> > >> >>
> > >>
> > >>
> >
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
> > >> >>iter.java:174)
> > >> >> at
> > >> >>
> > >>
> > >>
> >
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
> > >> >>iter.java:192)
> > >> >> at
> > >> >>
> > >>
> > >>
> >
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.buildDB(GoogDepsWr
> > >> >>iter.java:127)
> > >> >> at
> > >> >>
> > >>
> > >>
> >
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.getListOfFiles(Goo
> > >> >>gDepsWriter.java:79)
> > >> >> at
> > >> >>
> > >>
> > >>
> >
> >>org.apache.flex.compiler.internal.codegen.mxml.flexjs.MXMLFlexJSPublisher
> > >> >>.publish(MXMLFlexJSPublisher.java:325)
> > >> >> at
> > >> >>org.apache.flex.compiler.clients.MXMLJSC.compile(MXMLJSC.java:467)
> > >> >> at
> > >> >>
> > org.apache.flex.compiler.clients.MXMLJSC._mainNoExit(MXMLJSC.java:346)
> > >> >> at
> > >> >>
> org.apache.flex.compiler.clients.MXMLJSC.mainNoExit(MXMLJSC.java:272)
> > >> >> at
> > >> >>
> > >>
> > >>
> >
> >>org.apache.flex.compiler.clients.MXMLJSC.staticMainNoExit(MXMLJSC.java:23
> > >> >>1)
> > >> >> at
> > org.apache.flex.compiler.clients.MXMLJSC.main(MXMLJSC.java:177)
> > >> >>
> > >> >>
> > >> >> File not found: feathers.controls.SimpleButton
> > >> >>
> > >> >> If I put the SimpleButton class into the top-level package, it
> > compiles
> > >> >> without error. Any ideas?
> > >> >>
> > >> >> - Josh
> > >> >>
> > >>
> > >>
> >
>
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Michael Schmalle <te...@gmail.com>.
Josh,
Would it be easier for you to just put what you have changed in if()
statements and test whether the output type is JSC, then commit your
changes?
It's kind of hackish but then you would be back in sync. I really don't see
a problem with the right now until Alex is ready to change things.
Mike
On Tue, Jul 14, 2015 at 4:38 PM, Josh Tynjala <jo...@gmail.com> wrote:
> If anyone else is interested in using JSC, here are the changes that I
> made.
>
> In the getFilePath() function of GoogDepsWriter, on line 377:
>
> String classPath = className.replace(".", File.separator);
> if(classPath.equals(className))
> {
> classPath = className.replace("_", File.separator);
> }
>
> As a fallback, I kept the case where _ is replaced because the
> org_apache_flex_utils_Language is still named with underscores right now,
> and I was getting an error because of it.
>
> Later in the same function:
>
> String fwdClassPath = className.replace(".", "/");
> if(fwdClassPath.equals(className))
> {
> fwdClassPath = className.replace("_", "/");
> }
> String bckClassPath = className.replace(".", "\\");
> if(bckClassPath.equals(className))
> {
> bckClassPath = className.replace("_", "\\");
> }
>
> Finally, this change should also happen in the formatQualifiedName()
> function in the JSFlexJSDocEmitter class. The doc comments also currently
> reference class names with underscores.
>
> protected String formatQualifiedName(String name)
> {
> if (name.contains("goog.") || name.startsWith("Vector."))
> return name;
> name = name.replaceAll("\\.", ".");
> return name;
> }
>
> - Josh
>
> On Sat, Jul 11, 2015 at 10:22 AM, Josh Tynjala <jo...@gmail.com>
> wrote:
>
> > No rush! I can make the change locally.
> >
> > - Josh
> > On Jul 10, 2015 9:26 PM, "Alex Harui" <ah...@adobe.com> wrote:
> >
> >> Sorry for the late reply. I had internet issues all day.
> >>
> >> The problem is that GoogDepsWriter.java is still assuming “_” for
> package
> >> separators. See line 377.
> >> String classPath = className.replace("_", File.separator);
> >>
> >> I wasn’t planning on switching back to “.” until after next week or so,
> >> but I suppose I could shift priorities. Or if you have built your own
> >> compiler you can limp along by making this change in your local copy.
> >>
> >> -Alex
> >>
> >> On 7/10/15, 4:15 PM, "Josh Tynjala" <jo...@gmail.com> wrote:
> >>
> >> >Just in case someone gets confused, this line should not include
> >> >newPack.OtherClass:
> >> >
> >> >java.lang.RuntimeException: Unable to find JavaScript filePath for
> class:
> >> >newPack.OtherClass
> >> >
> >> >Bad copy-paste job. The real output was feathers.controls.SimpleButton
> >> >too.
> >> >
> >> >On Fri, Jul 10, 2015 at 4:06 PM, Josh Tynjala <jo...@gmail.com>
> >> >wrote:
> >> >
> >> >> It looks like the jsc output type is having trouble with classes when
> >> >> they're in packages. See the error below:
> >> >>
> >> >> Could not find file for class: feathers.controls.SimpleButton
> >> >> java.lang.RuntimeException: Unable to find JavaScript filePath for
> >> >>class:
> >> >> newPack.OtherClass
> >> >> at
> >> >>
> >>
> >>
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
> >> >>iter.java:174)
> >> >> at
> >> >>
> >>
> >>
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
> >> >>iter.java:192)
> >> >> at
> >> >>
> >>
> >>
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.buildDB(GoogDepsWr
> >> >>iter.java:127)
> >> >> at
> >> >>
> >>
> >>
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.getListOfFiles(Goo
> >> >>gDepsWriter.java:79)
> >> >> at
> >> >>
> >>
> >>
> >>org.apache.flex.compiler.internal.codegen.mxml.flexjs.MXMLFlexJSPublisher
> >> >>.publish(MXMLFlexJSPublisher.java:325)
> >> >> at
> >> >>org.apache.flex.compiler.clients.MXMLJSC.compile(MXMLJSC.java:467)
> >> >> at
> >> >>
> org.apache.flex.compiler.clients.MXMLJSC._mainNoExit(MXMLJSC.java:346)
> >> >> at
> >> >> org.apache.flex.compiler.clients.MXMLJSC.mainNoExit(MXMLJSC.java:272)
> >> >> at
> >> >>
> >>
> >>
> >>org.apache.flex.compiler.clients.MXMLJSC.staticMainNoExit(MXMLJSC.java:23
> >> >>1)
> >> >> at
> org.apache.flex.compiler.clients.MXMLJSC.main(MXMLJSC.java:177)
> >> >>
> >> >>
> >> >> File not found: feathers.controls.SimpleButton
> >> >>
> >> >> If I put the SimpleButton class into the top-level package, it
> compiles
> >> >> without error. Any ideas?
> >> >>
> >> >> - Josh
> >> >>
> >>
> >>
>
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Josh Tynjala <jo...@gmail.com>.
If anyone else is interested in using JSC, here are the changes that I made.
In the getFilePath() function of GoogDepsWriter, on line 377:
String classPath = className.replace(".", File.separator);
if(classPath.equals(className))
{
classPath = className.replace("_", File.separator);
}
As a fallback, I kept the case where _ is replaced because the
org_apache_flex_utils_Language is still named with underscores right now,
and I was getting an error because of it.
Later in the same function:
String fwdClassPath = className.replace(".", "/");
if(fwdClassPath.equals(className))
{
fwdClassPath = className.replace("_", "/");
}
String bckClassPath = className.replace(".", "\\");
if(bckClassPath.equals(className))
{
bckClassPath = className.replace("_", "\\");
}
Finally, this change should also happen in the formatQualifiedName()
function in the JSFlexJSDocEmitter class. The doc comments also currently
reference class names with underscores.
protected String formatQualifiedName(String name)
{
if (name.contains("goog.") || name.startsWith("Vector."))
return name;
name = name.replaceAll("\\.", ".");
return name;
}
- Josh
On Sat, Jul 11, 2015 at 10:22 AM, Josh Tynjala <jo...@gmail.com>
wrote:
> No rush! I can make the change locally.
>
> - Josh
> On Jul 10, 2015 9:26 PM, "Alex Harui" <ah...@adobe.com> wrote:
>
>> Sorry for the late reply. I had internet issues all day.
>>
>> The problem is that GoogDepsWriter.java is still assuming “_” for package
>> separators. See line 377.
>> String classPath = className.replace("_", File.separator);
>>
>> I wasn’t planning on switching back to “.” until after next week or so,
>> but I suppose I could shift priorities. Or if you have built your own
>> compiler you can limp along by making this change in your local copy.
>>
>> -Alex
>>
>> On 7/10/15, 4:15 PM, "Josh Tynjala" <jo...@gmail.com> wrote:
>>
>> >Just in case someone gets confused, this line should not include
>> >newPack.OtherClass:
>> >
>> >java.lang.RuntimeException: Unable to find JavaScript filePath for class:
>> >newPack.OtherClass
>> >
>> >Bad copy-paste job. The real output was feathers.controls.SimpleButton
>> >too.
>> >
>> >On Fri, Jul 10, 2015 at 4:06 PM, Josh Tynjala <jo...@gmail.com>
>> >wrote:
>> >
>> >> It looks like the jsc output type is having trouble with classes when
>> >> they're in packages. See the error below:
>> >>
>> >> Could not find file for class: feathers.controls.SimpleButton
>> >> java.lang.RuntimeException: Unable to find JavaScript filePath for
>> >>class:
>> >> newPack.OtherClass
>> >> at
>> >>
>>
>> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
>> >>iter.java:174)
>> >> at
>> >>
>>
>> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
>> >>iter.java:192)
>> >> at
>> >>
>>
>> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.buildDB(GoogDepsWr
>> >>iter.java:127)
>> >> at
>> >>
>>
>> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.getListOfFiles(Goo
>> >>gDepsWriter.java:79)
>> >> at
>> >>
>>
>> >>org.apache.flex.compiler.internal.codegen.mxml.flexjs.MXMLFlexJSPublisher
>> >>.publish(MXMLFlexJSPublisher.java:325)
>> >> at
>> >>org.apache.flex.compiler.clients.MXMLJSC.compile(MXMLJSC.java:467)
>> >> at
>> >> org.apache.flex.compiler.clients.MXMLJSC._mainNoExit(MXMLJSC.java:346)
>> >> at
>> >> org.apache.flex.compiler.clients.MXMLJSC.mainNoExit(MXMLJSC.java:272)
>> >> at
>> >>
>>
>> >>org.apache.flex.compiler.clients.MXMLJSC.staticMainNoExit(MXMLJSC.java:23
>> >>1)
>> >> at org.apache.flex.compiler.clients.MXMLJSC.main(MXMLJSC.java:177)
>> >>
>> >>
>> >> File not found: feathers.controls.SimpleButton
>> >>
>> >> If I put the SimpleButton class into the top-level package, it compiles
>> >> without error. Any ideas?
>> >>
>> >> - Josh
>> >>
>>
>>
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Josh Tynjala <jo...@gmail.com>.
No rush! I can make the change locally.
- Josh
On Jul 10, 2015 9:26 PM, "Alex Harui" <ah...@adobe.com> wrote:
> Sorry for the late reply. I had internet issues all day.
>
> The problem is that GoogDepsWriter.java is still assuming “_” for package
> separators. See line 377.
> String classPath = className.replace("_", File.separator);
>
> I wasn’t planning on switching back to “.” until after next week or so,
> but I suppose I could shift priorities. Or if you have built your own
> compiler you can limp along by making this change in your local copy.
>
> -Alex
>
> On 7/10/15, 4:15 PM, "Josh Tynjala" <jo...@gmail.com> wrote:
>
> >Just in case someone gets confused, this line should not include
> >newPack.OtherClass:
> >
> >java.lang.RuntimeException: Unable to find JavaScript filePath for class:
> >newPack.OtherClass
> >
> >Bad copy-paste job. The real output was feathers.controls.SimpleButton
> >too.
> >
> >On Fri, Jul 10, 2015 at 4:06 PM, Josh Tynjala <jo...@gmail.com>
> >wrote:
> >
> >> It looks like the jsc output type is having trouble with classes when
> >> they're in packages. See the error below:
> >>
> >> Could not find file for class: feathers.controls.SimpleButton
> >> java.lang.RuntimeException: Unable to find JavaScript filePath for
> >>class:
> >> newPack.OtherClass
> >> at
> >>
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
> >>iter.java:174)
> >> at
> >>
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
> >>iter.java:192)
> >> at
> >>
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.buildDB(GoogDepsWr
> >>iter.java:127)
> >> at
> >>
> >>org.apache.flex.compiler.internal.graph.GoogDepsWriter.getListOfFiles(Goo
> >>gDepsWriter.java:79)
> >> at
> >>
> >>org.apache.flex.compiler.internal.codegen.mxml.flexjs.MXMLFlexJSPublisher
> >>.publish(MXMLFlexJSPublisher.java:325)
> >> at
> >>org.apache.flex.compiler.clients.MXMLJSC.compile(MXMLJSC.java:467)
> >> at
> >> org.apache.flex.compiler.clients.MXMLJSC._mainNoExit(MXMLJSC.java:346)
> >> at
> >> org.apache.flex.compiler.clients.MXMLJSC.mainNoExit(MXMLJSC.java:272)
> >> at
> >>
> >>org.apache.flex.compiler.clients.MXMLJSC.staticMainNoExit(MXMLJSC.java:23
> >>1)
> >> at org.apache.flex.compiler.clients.MXMLJSC.main(MXMLJSC.java:177)
> >>
> >>
> >> File not found: feathers.controls.SimpleButton
> >>
> >> If I put the SimpleButton class into the top-level package, it compiles
> >> without error. Any ideas?
> >>
> >> - Josh
> >>
>
>
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Alex Harui <ah...@adobe.com>.
Sorry for the late reply. I had internet issues all day.
The problem is that GoogDepsWriter.java is still assuming “_” for package
separators. See line 377.
String classPath = className.replace("_", File.separator);
I wasn’t planning on switching back to “.” until after next week or so,
but I suppose I could shift priorities. Or if you have built your own
compiler you can limp along by making this change in your local copy.
-Alex
On 7/10/15, 4:15 PM, "Josh Tynjala" <jo...@gmail.com> wrote:
>Just in case someone gets confused, this line should not include
>newPack.OtherClass:
>
>java.lang.RuntimeException: Unable to find JavaScript filePath for class:
>newPack.OtherClass
>
>Bad copy-paste job. The real output was feathers.controls.SimpleButton
>too.
>
>On Fri, Jul 10, 2015 at 4:06 PM, Josh Tynjala <jo...@gmail.com>
>wrote:
>
>> It looks like the jsc output type is having trouble with classes when
>> they're in packages. See the error below:
>>
>> Could not find file for class: feathers.controls.SimpleButton
>> java.lang.RuntimeException: Unable to find JavaScript filePath for
>>class:
>> newPack.OtherClass
>> at
>>
>>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
>>iter.java:174)
>> at
>>
>>org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWr
>>iter.java:192)
>> at
>>
>>org.apache.flex.compiler.internal.graph.GoogDepsWriter.buildDB(GoogDepsWr
>>iter.java:127)
>> at
>>
>>org.apache.flex.compiler.internal.graph.GoogDepsWriter.getListOfFiles(Goo
>>gDepsWriter.java:79)
>> at
>>
>>org.apache.flex.compiler.internal.codegen.mxml.flexjs.MXMLFlexJSPublisher
>>.publish(MXMLFlexJSPublisher.java:325)
>> at
>>org.apache.flex.compiler.clients.MXMLJSC.compile(MXMLJSC.java:467)
>> at
>> org.apache.flex.compiler.clients.MXMLJSC._mainNoExit(MXMLJSC.java:346)
>> at
>> org.apache.flex.compiler.clients.MXMLJSC.mainNoExit(MXMLJSC.java:272)
>> at
>>
>>org.apache.flex.compiler.clients.MXMLJSC.staticMainNoExit(MXMLJSC.java:23
>>1)
>> at org.apache.flex.compiler.clients.MXMLJSC.main(MXMLJSC.java:177)
>>
>>
>> File not found: feathers.controls.SimpleButton
>>
>> If I put the SimpleButton class into the top-level package, it compiles
>> without error. Any ideas?
>>
>> - Josh
>>
Re: [FalconJX jsc] Compiler error when using classes in packages.
Posted by Josh Tynjala <jo...@gmail.com>.
Just in case someone gets confused, this line should not include
newPack.OtherClass:
java.lang.RuntimeException: Unable to find JavaScript filePath for class:
newPack.OtherClass
Bad copy-paste job. The real output was feathers.controls.SimpleButton too.
On Fri, Jul 10, 2015 at 4:06 PM, Josh Tynjala <jo...@gmail.com> wrote:
> It looks like the jsc output type is having trouble with classes when
> they're in packages. See the error below:
>
> Could not find file for class: feathers.controls.SimpleButton
> java.lang.RuntimeException: Unable to find JavaScript filePath for class:
> newPack.OtherClass
> at
> org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWriter.java:174)
> at
> org.apache.flex.compiler.internal.graph.GoogDepsWriter.addDeps(GoogDepsWriter.java:192)
> at
> org.apache.flex.compiler.internal.graph.GoogDepsWriter.buildDB(GoogDepsWriter.java:127)
> at
> org.apache.flex.compiler.internal.graph.GoogDepsWriter.getListOfFiles(GoogDepsWriter.java:79)
> at
> org.apache.flex.compiler.internal.codegen.mxml.flexjs.MXMLFlexJSPublisher.publish(MXMLFlexJSPublisher.java:325)
> at org.apache.flex.compiler.clients.MXMLJSC.compile(MXMLJSC.java:467)
> at
> org.apache.flex.compiler.clients.MXMLJSC._mainNoExit(MXMLJSC.java:346)
> at
> org.apache.flex.compiler.clients.MXMLJSC.mainNoExit(MXMLJSC.java:272)
> at
> org.apache.flex.compiler.clients.MXMLJSC.staticMainNoExit(MXMLJSC.java:231)
> at org.apache.flex.compiler.clients.MXMLJSC.main(MXMLJSC.java:177)
>
>
> File not found: feathers.controls.SimpleButton
>
> If I put the SimpleButton class into the top-level package, it compiles
> without error. Any ideas?
>
> - Josh
>