Opened 10 years ago

Closed 10 years ago

#90 closed enhancement (fixed)

review: Module descriptions

Reported by: jelte Owned by: jelte
Priority: low Milestone:
Component: Unclassified Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity:
Sub-Project: Feature Depending on Ticket:
Estimated Difficulty: Add Hours to Ticket:
Total Hours: Internal?:

Description

Currently, in bindctl, all modules are described as "Same here".
I think the best place to add a description is in the specification of that module's data and commands; i.e. add a value "module_description" below "module_name" in the .spec files (and support in module_spec.py/cc to read it, and make bindctl read out that value)

Subtickets

Change History (6)

comment:1 Changed 10 years ago by jelte

  • Owner set to jelte
  • Status changed from new to assigned

comment:2 Changed 10 years ago by jelte

  • Owner changed from jelte to UnAssigned
  • Summary changed from Module descriptions to review: Module descriptions

Ok, the change is in branches/trac90
Full diff:
svn diff -r 1624:HEAD branches/trac90

I had to update another feature for this addition to be used; b10-cmdctl now stores complete module specifications (it used to only store commands specifications and config_data specification separately)

Added 'review' to summary, as this feature may now be reviewed and merged.

comment:3 Changed 10 years ago by shane

  • Owner changed from UnAssigned to each

comment:4 Changed 10 years ago by each

  • Owner changed from each to jelte

Looks fine to me, except that you should probably also add descriptions to src/bin/auth/auth.pre.in and src/bin/xfrin/xfrin.pre.in (and maybe others I've missed).

comment:5 Changed 10 years ago by jelte

Ok, added some as-good-as-empty descriptions in r1849

Planning on merging tomorrow or thursday.

comment:6 Changed 10 years ago by jelte

  • Resolution set to fixed
  • Status changed from assigned to closed

Merged to trunk, r1856

Note: See TracTickets for help on using tickets.