trafodion icon indicating copy to clipboard operation
trafodion copied to clipboard

TRAFODION-2995 : modifying project and foundation links.

Open PierreSmits opened this issue 7 years ago • 18 comments

first batch of corrections. also included correction to download location of latest release (dyn/closer ref).

PierreSmits avatar Mar 17 '18 10:03 PierreSmits

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2490/

Traf-Jenkins avatar Mar 17 '18 10:03 Traf-Jenkins

Test Failed. https://jenkins.esgyn.com/job/Check-PR-master/2490/

Traf-Jenkins avatar Mar 17 '18 10:03 Traf-Jenkins

I can't see where the tests failed.

PierreSmits avatar Mar 17 '18 11:03 PierreSmits

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2491/

Traf-Jenkins avatar Mar 17 '18 12:03 Traf-Jenkins

Test Failed. https://jenkins.esgyn.com/job/Check-PR-master/2491/

Traf-Jenkins avatar Mar 17 '18 12:03 Traf-Jenkins

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2492/

Traf-Jenkins avatar Mar 17 '18 12:03 Traf-Jenkins

Test Failed. https://jenkins.esgyn.com/job/Check-PR-master/2492/

Traf-Jenkins avatar Mar 17 '18 12:03 Traf-Jenkins

https://jenkins.esgyn.com/job/static-checks/2849/consoleFull

2018-03-17 12:27:41 Unapproved licenses: 2018-03-17 12:27:41 2018-03-17 12:27:41 trafodion/Makefile 2018-03-17 12:27:41 trafodion/doap.rdf 2018-03-17 12:27:41 trafodion/env.sh 2018-03-17 12:27:41 trafodion/pom.xml 2018-03-17 12:27:41 trafodion/core/sql/common/11_ut2l.h 2018-03-17 12:27:41 trafodion/core/sql/common/1n_lt2u.h 2018-03-17 12:27:41 trafodion/core/sql/eh/00-readme 2018-03-17 12:27:41 trafodion/core/sql/export/AbortCallBack.h 2018-03-17 12:27:41 trafodion/docs/lob_guide/src/asciidoc/_chapters/about.adoc 2018-03-17 12:27:41 trafodion/docs/messages_guide/src/asciidoc/_chapters/about.adoc 2018-03-17 12:27:41 trafodion/docs/odb_user/src/asciidoc/_chapters/a_warnings.adoc

RAT apparently does not like https in the license statements.

svarnau avatar Mar 18 '18 03:03 svarnau

Hmm.

Is that something we can fix in our RAT configurations?

PierreSmits avatar Mar 18 '18 07:03 PierreSmits

I thought maybe it would be addressed in a later version of RAT than what we use, but I see no indication of that in their docs. Maybe we have to stick with exact license text, even for the old http URL.

svarnau avatar Mar 19 '18 06:03 svarnau

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2494/

Traf-Jenkins avatar Mar 19 '18 07:03 Traf-Jenkins

Test Failed. https://jenkins.esgyn.com/job/Check-PR-master/2494/

Traf-Jenkins avatar Mar 19 '18 07:03 Traf-Jenkins

jenkins, retest

DaveBirdsall avatar Apr 09 '18 18:04 DaveBirdsall

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2548/

Traf-Jenkins avatar Apr 09 '18 18:04 Traf-Jenkins

Test Failed. https://jenkins.esgyn.com/job/Check-PR-master/2548/

Traf-Jenkins avatar Apr 09 '18 18:04 Traf-Jenkins

Just checking up on this pull request to see if we can move it along. It looks like the only issue remaining is to change "https" back to "http" in the Apache license headers. @svarnau and @PierreSmits, do you agree?

DaveBirdsall avatar Apr 10 '18 22:04 DaveBirdsall

It seems to me that having either https or http in the licence header should not invalidate the test.

PierreSmits avatar Apr 11 '18 08:04 PierreSmits

We are just going by what RAT reports. There has been some discussion of this on the legel-discuss mail list, and it was suggested that even changing http to https in the license text is a change that would need to be dicussed and approved by the ASF.

svarnau avatar Apr 11 '18 16:04 svarnau