Opened 3 years ago
Closed 2 years ago
#4210 closed enhancement (complete)
flex and bison regen
| Reported by: | fdupont | Owned by: | fdupont |
|---|---|---|---|
| Priority: | low | Milestone: | Outstanding Tasks |
| Component: | classification | Version: | git |
| Keywords: | Cc: | ||
| CVSS Scoring: | Parent Tickets: | ||
| Sensitive: | no | Defect Severity: | N/A |
| Sub-Project: | DHCP | Feature Depending on Ticket: | |
| Estimated Difficulty: | 0 | Add Hours to Ticket: | 0 |
| Total Hours: | 0 | Internal?: | no |
Description (last modified by fdupont)
We can't rely on git merge to correctly regenerate files from flex and bison. I suggest to create a short script which:
- run bison and flex on a common box (*)
- add a timestamp in *.hh files (* *)
(*) I asked the upgrade/install of flex 2.5.39 and bison 3.0.4 on docs.isc.org
(* *) to make happy when configured with --enable-generate-parser. Note git does not update unchanged files so something must change so the idea of a timestamp.
Note this belongs to release engineering so even it is formally a 1.0 item it has (very?) low priority.
Subtickets
Attachments (1)
Change History (8)
comment:1 Changed 3 years ago by fdupont
- Description modified (diff)
comment:2 Changed 3 years ago by fdupont
comment:3 Changed 3 years ago by tomek
- Milestone changed from Kea-proposed to Outstanding Tasks
Moving all bugs to outstanding, we'll triage them with other defects.
comment:4 Changed 3 years ago by fdupont
Got a final answer from Dan: we should install our lex/bison. So this ticket can be unstalled and the discussion resumed.
comment:5 Changed 3 years ago by fdupont
- Component changed from Unclassified to classification
- Owner set to fdupont
- Status changed from new to accepted
- Type changed from defect to enhancement
Installed last flex and bison, wrapped bison with a script added a // Generated %Y%m%d%H%M comment at the beginning of .hh files in order to make them different and not ignore by git on commits.
Note docs has no log4cplus (and perhaps more) so ./configure --enable-generate-parser fails on it.
comment:6 Changed 3 years ago by fdupont
Everything ready. I propose to add something in the developer's guide and to attach the wrapper script to this ticket.
comment:7 Changed 2 years ago by fdupont
- Resolution set to complete
- Status changed from accepted to closed
script attached!
According to Dan it is not so easy to install bison 3 on docs. Stall this ticket...