[tor-bugs] #32798 [Core Tor/Tor]: Compile every header by itself as part of "make check"

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu Dec 19 08:36:06 UTC 2019


#32798: Compile every header by itself as part of "make check"
--------------------------+------------------------------------
 Reporter:  teor          |          Owner:  (none)
     Type:  task          |         Status:  needs_review
 Priority:  Medium        |      Milestone:  Tor: 0.4.3.x-final
Component:  Core Tor/Tor  |        Version:
 Severity:  Normal        |     Resolution:
 Keywords:                |  Actual Points:  1
Parent ID:  #32764        |         Points:  2
 Reviewer:  nickm, teor   |        Sponsor:
--------------------------+------------------------------------
Changes (by teor):

 * status:  new => needs_review
 * reviewer:   => nickm, teor
 * actualpoints:   => 1


Comment:

 I think I have this script in a reviewable, production-ready state. It
 checks the header failures against an exceptions file, and only fails when
 a header unexpectedly fails.

 See my PR, based on yours:
 * master: https://github.com/torproject/tor/pull/1618

 Here are the key changes:
   * Add an exceptions file, so we can make changes incrementally
   * Support out-of-tree builds
   * Look for more defines, more reliably
   * Check more headers
   * Other minor fixes
   * Create prefix.h for allowed prefix includes

 Maybe you can help with this change:
   * use CFLAGS etc. from the Makefile

 After we've finished reviews and revisions, we should:
   * rebase on #32764
     * and run "make update-build-headers-exceptions"

 There's still plenty we can do to improve it:
   * prefix.h:
     * stop including torint.h and testsupport.h
   * integrate using an include.am file, rather than generating a Makefile?
     * make Makefile dependencies actually work, rather than re-compiling
 every time

 I used 1 point on this task today, can you add what you used yesterday?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/32798#comment:3>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list