Opened 17 years ago
Closed 17 years ago
#244 closed defect (fixed)
bindings should use "swig -MD"
| Reported by: | Olly Betts | Owned by: | Richard Boulton |
|---|---|---|---|
| Priority: | high | Milestone: | 1.1.0 |
| Component: | Xapian-bindings | Version: | SVN trunk |
| Severity: | normal | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Operating System: | All |
Description (last modified by )
SWIG supports automatic generation of dependencies via -MD, etc options similar to those which GCC supports. The bindings should use this to ease development - then if you modify a xapian-core header which SWIG parses directly, make in the bindings will know to rerun SWIG.
Attachments (1)
Change History (9)
comment:1 by , 17 years ago
| Blocking: | 160 added |
|---|
comment:2 by , 17 years ago
| Operating System: | → All |
|---|---|
| Status: | new → assigned |
comment:4 by , 17 years ago
| Description: | modified (diff) |
|---|---|
| Milestone: | → 1.1 |
comment:5 by , 17 years ago
| Blocking: | 160 removed |
|---|
comment:6 by , 17 years ago
| Owner: | changed from to |
|---|---|
| Status: | assigned → new |
by , 17 years ago
| Attachment: | xapian-config-kill-swig-headers.patch added |
|---|
Combined patch to remove --swig-headers
comment:7 by , 17 years ago
| Description: | modified (diff) |
|---|---|
| Milestone: | 1.1.0 |
In fact, I'm just going to back out the --swigheaders change, and unset the milestone on this bug. Once we release a xapian-config with this option, we have to worry about breaking compatibility if we remove it again, so we definitely need to resolve that part for 1.1.0.
The second part - using -MD instead - is less urgent I think. Nobody has complained about the lack of such dependencies in release versions - these dependencies are really only useful if you're developing and so might modify xapian-core. And there's an easy workaround - just: touch xapian.i
comment:8 by , 17 years ago
| Priority: | normal → high |
|---|---|
| Status: | new → assigned |
This is currently breaking buildbot (or, at least, the buildbot builders which just do svn update then make, which is the majority of them). I could fix it manually for each builder (by doing "touch xapian.i"), but that's tedious and it would just break with the next change to the bindings anyway.
I agree that this doesn't need to block a release, but we do need to have a working buildbot to help ensure that code which is released compiled in different environments, so I'm increasing the priority of this bug so I don't forget to address it soon.
comment:9 by , 17 years ago
| Milestone: | → 1.1.0 |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Completed on SVN HEAD. Bindings now use -MD to get dependencies.
We've not yet made a release with --swigheaders in so we should address this for 1.1 so we don't need to worry about compatibility between releases with/without support for --swigheaders.