[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #21145 [Metrics/ExoneraTor]: Adapt ExoneraTor to using metrics-base



#21145: Adapt ExoneraTor to using metrics-base
--------------------------------+------------------------------
 Reporter:  iwakeh              |          Owner:  iwakeh
     Type:  enhancement         |         Status:  needs_review
 Priority:  Medium              |      Milestone:
Component:  Metrics/ExoneraTor  |        Version:
 Severity:  Normal              |     Resolution:
 Keywords:  metrics-2017        |  Actual Points:
Parent ID:                      |         Points:
 Reviewer:                      |        Sponsor:
--------------------------------+------------------------------

Comment (by karsten):

 Works! Pushed the metrics-base commit.

 But before I squash and push the ExoneraTor commits, did you make the
 following change on purpose?

 {{{
 -    <include name="postgresql-9.4.1212.jar"/>
 +    <include name="postgresql-jdbc3.jar"/>
 }}}

 I know that Debian stretch ships the JAR file as `postgresql.jar` together
 with three symbolic links:

 {{{
 lrwxrwxrwx 1 root root     14 Jan  9  2017
 /usr/share/java/postgresql-9.4.1212.jar -> postgresql.jar
 -rw-r--r-- 1 root root 650839 Jan  9  2017 /usr/share/java/postgresql.jar
 lrwxrwxrwx 1 root root     14 Jan  9  2017 /usr/share/java/postgresql-
 jdbc3.jar -> postgresql.jar
 lrwxrwxrwx 1 root root     14 Jan  9  2017 /usr/share/java/postgresql-
 jdbc4.jar -> postgresql.jar
 }}}

 But why pick the JDBC 3 link here rather than the one with the version
 number? That's different from what we do elsewhere.

 If this change just slipped in, I'll undo it and merge the rest.

 But if you intended to make this change, let's first discuss what names we
 should be using in build files in all metrics code bases.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21145#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs