Opened 9 years ago
Closed 9 years ago
#676 closed defect (fixed)
Enable caching in xapian-config by fixing typo
| Reported by: | Ryan Schmidt | Owned by: | Olly Betts |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.2.21 |
| Component: | Build system | Version: | 1.2.20 |
| Severity: | minor | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Operating System: | All |
Description
Hello, I believe there is a typo in xapian-config such that the intended caching behavior of the is_uninstalled function is never actually used, and the value is recomputed each time the function is called. I believe the attached patch fixes this.
Attachments (1)
Change History (3)
by , 9 years ago
| Attachment: | xapian-config.in.diff added |
|---|
comment:1 by , 9 years ago
| Component: | Other → Build system |
|---|---|
| Milestone: | → 1.2.21 |
| Status: | new → assigned |
comment:2 by , 9 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Backported for 1.2.21 in [6fa637d8b3200398944b075f9a00d8a1de677431].
Note:
See TracTickets
for help on using tickets.
Thanks, applied to git master in [716a2f331b9e34351b976cc680892d76c76e07c1/git].
Marking for backporting as it's a safe change, though I doubt the caching saves much time as it only avoids a
test -f. But if we're caching an answer, we ought to use it!