Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] Formatting and EPLv2

Which patch? There should not be changes to any other files as that would imply that those files are already checked in with bad formatting or other problems. What kind of changes are you seeing?

One possibility is that we are seeing differences with Eclipse versions. Which version of Eclipse are you using (passing to the script)?

I created a job that checks master is "clean" - This should verify that the files are clean on the build machine before you start. 


Jonah Graham
Kichwa Coders Ltd.

On Sat, 24 Nov 2018 at 19:49, Nathan Ridge <zeratul976@xxxxxxxxxxx> wrote:
I tried this, and the script made changes to a bunch of other source files, in addition to the ones my patch modifies. Should I be comitting those changes as well?

This is with my patch rebased onto the master branch, so it seems to me that those other files should already be formatted correctly.


From: Jonah Graham <jonah@xxxxxxxxxxxxxxxx>
Sent: November 23, 2018 6:06 PM
To: Nathan Ridge
Cc: CDT General developers list.
Subject: Re: [cdt-dev] Formatting and EPLv2

Yes, you can do that.

Run releng/scripts/<https:"" r="" #="" c="" 132975="" 4="" releng="" scripts="""">

You need an ECLIPSE environment variable that points at the eclipse install to use. It defaults to ~/buildtools/eclipse-SDK-4.9/eclipse (the value that works on the build machine). Once the script runs it will leave you to stage/commit any changes it has made.

Jonah Graham
Kichwa Coders Ltd.<http:"">

On Fri, 23 Nov 2018 at 18:01, Nathan Ridge <zeratul976@xxxxxxxxxxx<mailto:zeratul976@xxxxxxxxxxx>&gt; wrote:
Hi Jonah,

&gt; There is now a CI job to check code formatting and a few other rules. The job is and it runs which has some rules in it.
&gt; It means that gerrit CI check will fail if you have formatting/whitespace issues. Please let me know what you think.

Is there a local command I can run to reformat my code (which perhaps has an old patch applied) to satisfy the check?


Back to the top