Opened 3 years ago

Last modified 3 years ago

#6917 assigned defect

RFE: Ensure bundles have the same source and installed names

Reported by: Zach Pearson Owned by: Greg Couch
Priority: moderate Milestone:
Component: Build System Version:
Keywords: Cc: chimerax-programmers
Blocked By: Blocking:
Notify when closed: Platform: all
Project: ChimeraX

Description

Having different names for source folders vs. installed folders breaks most modern programming tooling, which expects a 1:1 correspondence between the source module name and the installed module name. It's a little sand in the gears, breaking among other things IDEs, linters, and automated testers.

AFAIK the only affected bundle is seq_align, which is named alignments in the source code.

Change History (7)

comment:1 by Zach Pearson, 3 years ago

Priority: blocker → moderate

comment:2 by Eric Pettersen, 3 years ago

It also affects at least alignment_hdrs (alignment_headers) and nih-presets (nih_presets).

comment:3 by Zach Pearson, 3 years ago

Owner: changed from Zach Pearson to Eric Pettersen

I don't own any of the affected code so I'm neutral to what the names end up being. Reassigning to you because you're in the git blame for all 3 the most :)

comment:4 by Eric Pettersen, 3 years ago

So, went through and actually checked for all bundles whose source directory isn't the same as their module name. They are:

Greg:

linux_support→linux
buttons→buttonpanel

Zach(!):

task_manager→taskmanager

Eric:

rotamer_lib_mgr→rotamers
alignment_hdrs→alignment_headers
alignments→seqalign
nih-presets→nih_presets
seq_view→seq-view(!)
align_algs→alignment_algs

in reply to:  5 ; comment:5 by Zach Pearson, 3 years ago 

😅

comment:7 by Eric Pettersen, 3 years ago

Owner: changed from Eric Pettersen to Greg Couch

Okay, I've made my module and source directory names match, so reassigning to Greg since he's the only one left with naming mismatches.

Note: See TracTickets for help on using tickets.