Ticket #226 (closed defect: fixed)

Opened 10 months ago

Last modified 6 months ago

Distcheck is broken due to missing rule for documentation files

Reported by: richard Owned by: richard
Priority: normal Milestone: 1.0.7
Component: Build system Version: SVN trunk
Severity: normal Keywords:
Cc: olly Blocked By:
Operating System: All Blocking:

Description (last modified by richard) (diff)

If I run "make distcheck", the "distdir" step in the generated distribution fails for me:

make[1]: Entering directory `/home/richard/private/Working/xapian/working/build/xapian-core/docs' make[1]: *** No rule to make target apidoc/html/*.gif', needed by distdir'. Stop.

This appears to have been a problem since revision 9747: see the following link for the details of what changed there:

http://svn.xapian.org/trunk/xapian-core/docs/Makefile.am?r1=9650&r2=9747

The problem is that there is no rule to generate the *.gif files (or any of the other files in apidoc/html/ except for index.html). In a built tree, "make dist" succeeds because the files will be generated by earlier build steps, but in the distcheck tree these earlier build steps aren't raised.

I think the cleanest fix is probably to revert revision 9747, and instead make a dist-hook rule or something similar to clean up the copied directory in the target.

The complete error log leading up to this can be seen in the buildbot: eg, at http://localhost:8010/xapian_head_clean_linux_amd64/builds/156/step-distcheck/0 if you have the usual ssh tunnel to buildbot running.

Change History

Changed 10 months ago by olly

  • status changed from new to assigned

I guess we'd need a "find" invocation to delete any files in that appropriate subtree of the "dist tree" which don't have the correct extension. A bit ugly, but then a multi-target rule isn't great either.

Hmm, actually, perhaps we can just provide the rule to copy over into the "dist tree".

Are you happy to look at this? I'm a bit short of time...

Changed 10 months ago by richard

  • owner changed from newbugs to richard
  • status changed from assigned to new

I'll try and take a look at this tomorrow.

Changed 10 months ago by olly

  • cc olly@… added

Changed 10 months ago by richard

  • status changed from new to assigned

Changed 10 months ago by trac

  • platform set to All

Changed 7 months ago by richard

  • status changed from assigned to closed
  • resolution set to fixed
  • description modified (diff)
  • milestone set to 1.0.7

Fixed in changeset [10462]. (distcheck is still broken, but it gets past the problem with the documentation files).

Changed 6 months ago by olly

The above isn't clear, but this change *has* been backported to 1.0.7.

Note: See TracTickets for help on using tickets.