Hi Josh, I think if after next build someone notice this files changed we can commit that new versions as I assume is ok to do that. thanks
El mar., 3 dic. 2019 a las 18:38, Josh Tynjala (<[email protected]>) escribió: > I don't know why the Maven build would be modifying those two files. When > you committed the modifications, the contents of the files were not > changed. The only change was to the file permissions to make them > executable: > > > https://github.com/apache/royale-compiler/commit/f48d1051cd2524bc133f96081aa67ba55212bab0 > > Perhaps Maven tries to be helpful by detecting if something is a shell > script and ensures that it is executable. It seems strange that this would > happen in a src folder instead of when a file is copied to the target > folder, though. > > -- > Josh Tynjala > Bowler Hat LLC <https://bowlerhat.dev> > > > On Tue, Dec 3, 2019 at 9:14 AM Carlos Rovira <[email protected]> > wrote: > > > Hi ok. > > So what I don't understand is what each time I build from scratch the > build > > indicates that 2 files are modified. Should I commit the modifications? > and > > if you created it, is right that the build is updating those 2 files? > > > > thanks! > > > > El mar., 3 dic. 2019 a las 16:53, Josh Tynjala (< > [email protected] > > >) > > escribió: > > > > > Hi Carlos, > > > > > > I think that you are confused because you believe that these files are > > > generated, but they are not generated. About two weeks ago, I > personally > > > created these files in my own text editor. Here's my original commit > > where > > > I added them to the repository: > > > > > > > > > > > > https://github.com/apache/royale-compiler/commit/f0d9176c405c0508255d20a98b095ea21ff4b238 > > > > > > -- > > > Josh Tynjala > > > Bowler Hat LLC <https://bowlerhat.dev> > > > > > > > > > On Tue, Dec 3, 2019 at 7:18 AM Carlos Rovira <[email protected]> > > > wrote: > > > > > > > Hi Josh, > > > > just put them back. I'm a bit confused with this files, since we > should > > > not > > > > have generated files. But as well I can't add it to .gitignore list. > > > > What should we do with those? > > > > thanks > > > > > > > > El lun., 2 dic. 2019 a las 16:36, Josh Tynjala (< > > > [email protected] > > > > >) > > > > escribió: > > > > > > > > > These files are not actually generated. Please put them back. > > > > > > > > > > -- > > > > > Josh Tynjala > > > > > Bowler Hat LLC <https://bowlerhat.dev> > > > > > > > > > > > > > > > On Thu, Nov 28, 2019 at 8:56 AM Carlos Rovira < > > [email protected] > > > > > > > > > wrote: > > > > > > > > > > > I was figuring that, but wasn't sure, will remove it > > > > > > > > > > > > El jue., 28 nov. 2019 a las 17:04, Alex Harui > > > > (<[email protected] > > > > > >) > > > > > > escribió: > > > > > > > > > > > > > I haven't built the compiler, but in general we do not want to > > > commit > > > > > > > generated files. Why would these be an exception? > > > > > > > > > > > > > > -Alex > > > > > > > > > > > > > > On 11/28/19, 7:17 AM, "[email protected]" < > > > > > > [email protected]> > > > > > > > wrote: > > > > > > > > > > > > > > This is an automated email from the ASF dual-hosted git > > > > repository. > > > > > > > > > > > > > > carlosrovira pushed a commit to branch develop > > > > > > > in repository > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-compiler.git&data=02%7C01%7Caharui%40adobe.com%7Cbe827a0702184edabb1708d774162394%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637105510754711745&sdata=sOb8FVjrZzDwZD2wc1iELgaV6z3OhrxHgHYmmJep8BI%3D&reserved=0 > > > > > > > > > > > > > > > > > > > > > The following commit(s) were added to refs/heads/develop by > > > this > > > > > > push: > > > > > > > new f48d105 should we add these generated files? (if > > not > > > > > please > > > > > > > remove) > > > > > > > f48d105 is described below > > > > > > > > > > > > > > commit f48d1051cd2524bc133f96081aa67ba55212bab0 > > > > > > > Author: Carlos Rovira <[email protected]> > > > > > > > AuthorDate: Thu Nov 28 16:17:39 2019 +0100 > > > > > > > > > > > > > > should we add these generated files? (if not please > > remove) > > > > > > > --- > > > > > > > compiler-jx/src/assembly/compiler-override-scripts/compc > | 0 > > > > > > > compiler-jx/src/assembly/compiler-override-scripts/mxmlc > | 0 > > > > > > > 2 files changed, 0 insertions(+), 0 deletions(-) > > > > > > > > > > > > > > diff --git > > > > > a/compiler-jx/src/assembly/compiler-override-scripts/compc > > > > > > > b/compiler-jx/src/assembly/compiler-override-scripts/compc > > > > > > > old mode 100644 > > > > > > > new mode 100755 > > > > > > > diff --git > > > > > a/compiler-jx/src/assembly/compiler-override-scripts/mxmlc > > > > > > > b/compiler-jx/src/assembly/compiler-override-scripts/mxmlc > > > > > > > old mode 100644 > > > > > > > new mode 100755 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Carlos Rovira > > > > > > http://about.me/carlosrovira > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Carlos Rovira > > > > http://about.me/carlosrovira > > > > > > > > > > > > > -- > > Carlos Rovira > > http://about.me/carlosrovira > > > -- Carlos Rovira http://about.me/carlosrovira
