If we decided to stick with old indent, which i still disagree that is right thing to do, can you just compile indent all the time and modify path so /opt/vpp/…/bin/ comes first. I really don’t like one more option in the top level Makefile.
— Damjan > On 18 Apr 2020, at 10:29, Andrew Yourtchenko <ayour...@gmail.com> wrote: > > I made https://gerrit.fd.io/r/#/c/vpp/+/22963/ > <https://gerrit.fd.io/r/#/c/vpp/+/22963/> that you can try and see how it > works for you. > > It allows to install the “correct” version of indent into the build tree, so > the rest of the system is unaffected. > > --a > >> On 11 Apr 2020, at 14:04, Dave Barach via lists.fd.io <http://lists.fd.io/> >> <dbarach=cisco....@lists.fd.io <mailto:dbarach=cisco....@lists.fd.io>> wrote: >> >> >> The script works fine. You have the wrong version of gnu indent installed. >> This is the version you need: >> >> $ indent --version >> GNU indent 2.2.11 >> >> From: vpp-dev@lists.fd.io <mailto:vpp-dev@lists.fd.io> <vpp-dev@lists.fd.io >> <mailto:vpp-dev@lists.fd.io>> On Behalf Of Zhang Yuwei >> Sent: Saturday, April 11, 2020 1:04 AM >> To: vpp-dev@lists.fd.io <mailto:vpp-dev@lists.fd.io> >> Subject: [vpp-dev] Checkstyle script not work in ubuntu >> >> Hi Guys, >> I find checkstyle script doesn’t work normally in ubuntu >> sometimes that I run make fixstyle in ubuntu and submit the code to gerrit >> but still fail in checkstyle step. I need to move to centos to make it work, >> can anybody check this? Thanks a lot. >> >> Regards, >> Yuwei >> >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#16100): https://lists.fd.io/g/vpp-dev/message/16100 Mute This Topic: https://lists.fd.io/mt/72939086/21656 Group Owner: vpp-dev+ow...@lists.fd.io Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
