Mailing-List: contact dev-help@flex.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flex.apache.org
From: Christofer Dutz <christofer.dutz@c-ware.de>
To: "dev@flex.apache.org" <dev@flex.apache.org>
Subject: AW: AW: [FlexJS] Problem with some test cases (@export was changed to
 @expose)?
Thread-Topic: AW: [FlexJS] Problem with some test cases (@export was changed
 to @expose)?
Thread-Index: AQHSGRiUuZvGizr9bE2nxo3UcFkuNqCOBsOA//+WioCAANxQeIAAAcgAgAAD08uAABIQlw==
Date: Wed, 28 Sep 2016 08:41:39 +0000
Message-ID: <HE1PR0501MB24284FF38DADD60B0A56458FA2CF0@HE1PR0501MB2428.eurprd05.prod.outlook.com>
References: <D410512D.764A5%aharui@adobe.com>
 <CANDXkxD9YvOAXtoOnRF9YUJ27_n+NPrDwOPbDc4Z3KgeRRwkTA@mail.gmail.com>
 <D41061AF.764D2%aharui@adobe.com>
 <HE1PR0501MB2428D26B82C9BC46A4E838BFA2CF0@HE1PR0501MB2428.eurprd05.prod.outlook.com>,<CANDXkxAUePNLU0gzYBc=-FETYi42LshSjVC5HGJo6oDw+ubiXQ@mail.gmail.com>,<HE1PR0501MB2428ACF1127A3CC032DE0BF6A2CF0@HE1PR0501MB2428.eurprd05.prod.outlook.com>
In-Reply-To: <HE1PR0501MB2428ACF1127A3CC032DE0BF6A2CF0@HE1PR0501MB2428.eurprd05.prod.outlook.com>
Accept-Language: de-DE, en-US
Content-Language: de-DE
received-spf: None (protection.outlook.com: c-ware.de does not designate
 permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: multipart/alternative;
	boundary="_000_HE1PR0501MB24284FF38DADD60B0A56458FA2CF0HE1PR0501MB2428_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Sep 2016 08:41:39.5954
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 9d387546-1437-4b89-846c-691d64a7e74d
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0501MB2425
archived-at: Wed, 28 Sep 2016 08:41:55 -0000

--_000_HE1PR0501MB24284FF38DADD60B0A56458FA2CF0HE1PR0501MB2428_
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

Could one of you guys please have a look at what's going wrong?

I could track it back to the Framework expecting some different implementat=
ion of Class.as

This seems to be generated in the js externs module ... I could track it ba=
ck to being generated by the missing.js (line 120ff)


I have no clue at what's going wrong here ... would be more than great if y=
ou could have a look at it.


Chris

________________________________
Von: Christofer Dutz <christofer.dutz@c-ware.de>
Gesendet: Mittwoch, 28. September 2016 09:20:52
An: dev@flex.apache.org
Betreff: AW: AW: [FlexJS] Problem with some test cases (@export was changed=
 to @expose)?

Currently the framework seems to have problems ... I noticed that, but I wa=
s a little too consumed with my duties ad TAC member to deal with it ... I =
better have a look as soon as possible.


Chris

________________________________
Von: Greg Dove <greg.dove@gmail.com>
Gesendet: Mittwoch, 28. September 2016 09:06:12
An: dev@flex.apache.org
Betreff: Re: AW: [FlexJS] Problem with some test cases (@export was changed=
 to @expose)?

Chris, I was seeing something not working with Maven in typedefs, a patch
not being applied cleanly.
I did not see this with ant. Justin also checked and confirmed this.
Not sure if another url has changed?


On Wed, Sep 28, 2016 at 8:03 PM, Christofer Dutz <christofer.dutz@c-ware.de=
>
wrote:

> Well the Maven build went back to blue ... thanks for fixing :-)
>
>
> I guess as I finished my work on the site deployment so the build works
> again. It's always worth to have a look at the official ASF Jenkins:
>
>
> https://builds.apache.org/view/E-G/view/Flex/
>
>
> Even if it builds with Maven, it does almost all of the tests and you
> should get feedback far more frequent as when using our external build
> system. And stuff that works with Ant and doesn't with Maven should be
> considered an error that needs further addressing.
>
>
> Chris
>
> ________________________________
> Von: Alex Harui <aharui@adobe.com>
> Gesendet: Mittwoch, 28. September 2016 02:51:23
> An: dev@flex.apache.org
> Betreff: Re: AW: [FlexJS] Problem with some test cases (@export was
> changed to @expose)?
>
> Right now, the flex_sdk Ant CI build is failing because a third-party
> download is unavailable.  Once that comes back, flex_falcon will be built
> again and hopefully it will all work.  Probably will take several hours t=
o
> shake out so don't spend any time thinking about it until after you see
> the flex_sdk build pass again.
>
> -Alex
>
> On 9/27/16, 5:08 PM, "Greg Dove" <greg.dove@gmail.com> wrote:
>
> >Alex, can you please let me know if this is related to something I did?
> >Is there something I need to fix?
> >I added some fixes this morning and checked that the maven build was
> >working locally.
> >But I don't know what I am doing that is different to the CI build.
> >
> >
> >
> >On Wed, Sep 28, 2016 at 12:40 PM, Alex Harui <aharui@adobe.com> wrote:
> >
> >> The CI Ant build now fails as well.  I just saw the email on the list.
> >> The CI server keeps getting stuck on flex_sdk_test so Falcon hadn't ru=
n
> >>in
> >> a couple of days.  The CI falcon build also runs the sdk.dependent.tes=
ts
> >> and flexjs.dependent.tests targets that don't run by default.
> >>
> >> -Alex
> >>
> >> On 9/27/16, 1:58 PM, "Christofer Dutz" <christofer.dutz@c-ware.de>
> >>wrote:
> >>
> >> >Hi Greg,
> >> >
> >> >
> >> >Well investigating this had me worry a little ... not the problem
> >>itself,
> >> >but the Ant build seems to be reporting that all is fine and I could
> >> >confirm the method being listed as successful in the build report.
> >> >
> >> >
> >> >But even if I run the test in IntelliJ without Maven, the test fails
> >>and
> >> >from having a look at it, it should fail. So the question remains: wh=
y
> >> >didn't the Ant build fail?
> >> >
> >> >
> >> >Chris
> >> >
> >> >________________________________
> >> >Von: Greg Dove <greg.dove@gmail.com>
> >> >Gesendet: Dienstag, 27. September 2016 22:46:27
> >> >An: dev@flex.apache.org
> >> >Betreff: Re: [FlexJS] Problem with some test cases (@export was chang=
ed
> >> >to @expose)?
> >> >
> >> >I wonder if I had something cached somewhere that needed an extra
> >>'clean'
> >> >-
> >> >I am still learning this, I guess.
> >> >
> >> >On Wed, Sep 28, 2016 at 9:43 AM, Greg Dove <greg.dove@gmail.com>
> wrote:
> >> >
> >> >> I am working through these now with the maven build, I found a few
> >>areas
> >> >> that needed attention - sorry.
> >> >> Yes I had expected this to work the same across the two builds as
> >>well.
> >> >>
> >> >>
> >> >>
> >> >> On Wed, Sep 28, 2016 at 9:42 AM, Christofer Dutz <
> >> >> christofer.dutz@c-ware.de> wrote:
> >> >>
> >> >>> Ok ... so I tracked down the problem to
> >> >>>
> >> >>> JSGoogDocEmitter
> >> >>>
> >> >>> there in line 344 in emitPublic, you seem to have changed the
> >>output.
> >> >>>
> >> >>> Should I adjust the testcase? I don't quite understand why the tes=
t
> >> >>> doesn't break in the Ant build, because the output should be wrong
> >> >>>there
> >> >>> too ...
> >> >>>
> >> >>>
> >> >>> Chris
> >> >>>
> >> >>>
> >> >>> ________________________________
> >> >>> Von: Greg Dove <greg.dove@gmail.com>
> >> >>> Gesendet: Dienstag, 27. September 2016 19:53:45
> >> >>> An: dev@flex.apache.org
> >> >>> Betreff: Re: [FlexJS] Problem with some test cases (@export was
> >>changed
> >> >>> to @expose)?
> >> >>>
> >> >>> This was me, and was needed for reflection support into static
> >>members.
> >> >>>
> >> >>> @expose is supposed to be deprecated, but currently seems to be th=
e
> >> >>>only
> >> >>> option that works for statics. Josh discovered this a while back I
> >> >>>think,
> >> >>> with static accessors.
> >> >>>
> >> >>> I will check this test now, I thought I had them all updated.
> >> >>>
> >> >>> cheers
> >> >>> Greg
> >> >>>
> >> >>>
> >> >>>
> >> >>> On Wed, Sep 28, 2016 at 3:46 AM, Christofer Dutz <
> >> >>> christofer.dutz@c-ware.de>
> >> >>> wrote:
> >> >>>
> >> >>> > Hi,
> >> >>> >
> >> >>> >
> >> >>> > it seems some recent changes broke things in the test-suite. I a=
m
> >> >>> getting
> >> >>> > failures in TestFlexJSClass.
> >> >>> >
> >> >>> > For:
> >> >>> >
> >> >>> >
> >> >>> > org.apache.flex.compiler.internal.codegen.js.flexjs.
> >> >>> > TestFlexJSClass#testConstants
> >> >>> >
> >> >>> >
> >> >>> > and
> >> >>> >
> >> >>> >
> >> >>> > org.apache.flex.compiler.internal.codegen.js.flexjs.
> >> >>> > TestFlexJSClass#testMethods
> >> >>> >
> >> >>> >
> >> >>> > it seems @export was changed to @expose ... would be cool if
> >>someone
> >> >>>who
> >> >>> > knows what happened here could fix that.
> >> >>> >
> >> >>> >
> >> >>> > Chris
> >> >>> >
> >> >>>
> >> >>
> >> >>
> >>
> >>
>
>

--_000_HE1PR0501MB24284FF38DADD60B0A56458FA2CF0HE1PR0501MB2428_--