Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#425 closed task (fixed)

Make svndigest to include plplot generated graphics in the output on our demo machine

Reported by: Jari Häkkinen Owned by: Jari Häkkinen
Priority: major Milestone: svndigest 0.8
Component: output Version: trunk
Keywords: Cc:

Description

This issue is not really a svndigest problem but rather an issue related to the fact that svndigest requires plplot 5.9.6 which is not released yet, and some lower level graphics support is missing on the host machine.

Change History (8)

comment:1 Changed 14 years ago by Peter Johansson

Owner: changed from Peter Johansson to Jari Häkkinen

comment:2 Changed 13 years ago by Peter Johansson

As I've mentioned before, if we are gonna produce output both from the latest stable branch and from trunk, it is crucial that we create those two reports in two different working copies. In other words we could have a file structure as follows:

svndigest-0.8
svndigest-trunk
svndigest-trunk-for-stable-report

The dependencies for when to run what becomes a bit complicated so I've recently translated my cron scripts to Makefiles and then I can just issue

(cd /path/to/pristine && make svndigest) > /dev/null

Typical rules are

MOVE_IF_CHANGE=svndigest-trunk/build_support/move-if-change
FORCE:

svndigest-trunk/svn-time-stamp: FORCE
  svn update svndigest-trunk
  svnversion svndigest-trunk > $@-t
  $(MOVE_IF_CHANGE) $@-t $@

In that way it becomes easy to manage the different dependencies and one could for example have a rule:

$(TARGET)/svndigest-0.8: svndigest-0.8/bin/svndigest svndigest-trunk-for-stable-report/svn-time-stamp
   ./svndigest-0.8/bin/svndigest <some options>

That, of course, assumes there is a rule for svndigest-0.8/bin/svndigest such as

svndigest-0.8/bin/svndigest: svndigest-0.8/svn-time-stamp
  ./bootstrap
  ./configure $(CONFIGURE_OPTIONS)
  make check

If you want I can translate the current shell script into a Makefile. My experience is that this is easier manage.

comment:3 Changed 13 years ago by Jari Häkkinen

Sure, go ahead.

comment:4 Changed 13 years ago by Jari Häkkinen

Status: newassigned

comment:5 Changed 13 years ago by Jari Häkkinen

Peter, thanks for the Makefile. It seems to work but due to ticket:451 I cannot generate reports. svndigest complains on WC not up to date if I remove the man directory from Makefile.am. If I keep man the tests fail. Waiting for resolution of ticket:451 or an install og help2man on the report generation machine (I have sent a request to Mattias).

comment:6 in reply to:  5 Changed 13 years ago by Peter Johansson

Replying to jari:

Peter, thanks for the Makefile. It seems to work but due to ticket:451 I cannot generate reports. svndigest complains on WC not up to date if I remove the man directory from Makefile.am. If I keep man the tests fail. Waiting for resolution of ticket:451 or an install og help2man on the report generation machine (I have sent a request to Mattias).

Hm, I see that one problem is that one wc has two roles: 1) first we build trunk version of svndigest there and 2) we run use it as --root for a svndigest run. If we had separated these two roles into two directories this would be less of a problem, in other words, perhaps we should remove the symbolic link in the file structure and rather checkout another wc. On the other hand, this is perhaps a rare case and hopefully the build will work in the future.

I have fixed ticket #451 and the build should (hopefully) work now. This is only in the stable though. In the trunk build I think you can bypass the problem by adding HELP2MAN=true to configure argument list.

comment:7 Changed 13 years ago by Jari Häkkinen

Resolution: fixed
Status: assignedclosed

comment:8 Changed 13 years ago by Jari Häkkinen

You'll find the output here http://cbbp.thep.lu.se/~jari/svndigest/

Note: See TracTickets for help on using tickets.