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
(* *) 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.


Attachments (1)

bison.script (252 bytes) - added by fdupont 2 years ago.
bison wrapper

Download all attachments as: .zip

Change History (8)

comment:1 Changed 3 years ago by fdupont

  • Description modified (diff)

comment:2 Changed 3 years ago by fdupont

According to Dan it is not so easy to install bison 3 on docs. Stall this ticket...

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.

Changed 2 years ago by fdupont

bison wrapper

comment:7 Changed 2 years ago by fdupont

  • Resolution set to complete
  • Status changed from accepted to closed

script attached!

Note: See TracTickets for help on using tickets.