Patrice Dumas wrote:
> Hello,
> 
> During the review for inclusion in fedora extras a reviewer found
> that some files of elektra had names that were to common, and could 
> lead to name clash. The issue is in fact very relevant for kdb,
> with kdb itself, kdb.h and kdb.1 kdb.3. 
> 
> I think that the include files should be in a directory, and therefore 
> end up in, say,
> /usr/include/elektra
> This should be changed without creating issues since elektra uses 
> pkgconfig. I think this really should be done. And kdb.h could have a 
> more relevant name like elektra-keys.h.
> 

+1 with the minor change that kdb.h should instead be called 
elektra/keys.h. That is, if the include directory is /usr/include it 
should be /usr/include/elektra/keys.h.

> 
> The kdb executable could also have its name changed to something 
> more relevant, and similarly with kdbd. Indeed these names don't 
> show that they are part of elektra. For example kdb could be 
> elektra-admin, or elektra-cmd, or something along those lines.
> and kdbd could be elektrad, or kdb-server.

Just a couple more suggestions:

    kdb -> elektra-tool        (maybe elektra-client?)
    kdbd -> elektra-server  (or maybe elektra-daemon?)

> 
> The changes in executable names will be much more painful, so it 
> should be considered carefully wether they are needed or not.

I don't have much say in this, but I think they ought to be done. There 
is no excuse for three and four-letter command names unless it is stuff 
that is used *all* the time. Shell scripts can use

     KDB=elektra-tool      (or whatever the final name ends up being)

     $KDB bla
     $KDB blabla
     ...

> 
> Symlinks or hardlinks could be used for a given number of releases 
> or some amount of time to keep backward compatibility.

+0. Don't we want the breakage sooner rather than later? Packagers can 
always add the necessary symlinks to their packages if they so desire.

Cheers,

-- 
Bardur Arantsson
<[EMAIL PROTECTED]>

- I tell ya, they only come out at night, or, in this case, the
daytime.
                               Chief Clancy Wiggum, 'The Simpsons'


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Registry-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/registry-list

Reply via email to