[Pki-devel] [PATCH] 190 Reorganized TPS templates and scripts.

[Ade Lee]

On Sun, 2012-12-02 at 20:55 -0600, Endi Sukma Dewata wrote:
> On 11/29/2012 11:59 PM, Ade Lee wrote:
> > Looks good - and is the right idea.  What makes you think that cfg.pl is
> > not used?  There are functions that are defined there that are used in
> > the esc cgi scripts.
> >
> > Please restore cfg.pl.
> 
> As discussed on IRC, there are 2 cfg.pl files, one in the TPS core page 
> and the other in the TPS theme package. In the original code before the 
> patch, the code in pkicreate:2064 copies the cfg.pl in theme to override 
> the cfg.pl in core:
> 
>    return 0 if !copy_directory($ui_subsystem_path, $pki_instance_path,
>        $default_dir_permissions, $default_file_permissions,
>        $pki_user, $pki_group);
> 
> So the original cfg.pl in core package was never actually used.
> 
> In the updated #192 this code no longer exists since the TPS theme has 
> being removed.
> 

ACK.

Whats troubling is that neither version appears to be correct.  The
correct version is in the 8.1 branch.  We don't have to fix this here.
You can commit your patch, but please open a ticket to fix the security
officer workstation cgi scripts in dogtag 10.