[TODO] config/gen/makefiles/dynpmc_pl.in sub compile_cmd()

[jkeenan]

The XXX flag comment below generated a t/codingstd/perlcritic.t failure tonight. (Why it did so tonight for flag comments that have been there for years? No idea.)

sub compile_cmd {
    my ($target, $source) = @_;

    my $dynpmc_include = '';
    if (defined $ENV{DYNPMC_INCLUDE} )
    {
        $dynpmc_include = $ENV{DYNPMC_INCLUDE};
        $dynpmc_include =~ s/,/$PATHQUOTE -I$PATHQUOTE/g;
        $dynpmc_include = "-I" . $PATHQUOTE . $dynpmc_include . $PATHQUOTE . " ";
    }


    # XXX Reference to BGC - need long term solution for passing in include paths
    return
        "$CC " .
        '@cc_o_out@' . $target . " " .
        "-I" . $PATHQUOTE . '@build_dir@@slash@include' . $PATHQUOTE . " " .
        "-I" . $PATHQUOTE . '@build_dir@@slash@src@slash@pmc' . $PATHQUOTE . " " .
        $dynpmc_include . " " .
        "$CFLAGS " .
        $source;
};

So I'm converting it to a ticket. Don't know what BGC is and I don't know why we need a long-term solution. Comments?

[jkeenan]

Was also reported in TT 290. Retaining this one as open.

[allison]

Deprecating the whole script. See TT #338.

[jkeenan]

Replying to allison:

Deprecating the whole script. See TT #338.

In a subsequent comment, Allison appears to have postponed the deprecation of tools/build/dynpmc_pl. So this issue remains open.

[jkeenan]

Replying to jkeenan:

In a subsequent comment, Allison appears to have postponed the deprecation of tools/build/dynpmc_pl. So this issue remains open.

Just to clarify: What we need to know is: What is wrong with the code beneath the comment? What is the problem with include paths? Why doesn't the cited code constitute a long-term solution?

Thank you very much.
kid51

[jkeenan]

> 
> Just to clarify:  What we need to know is:  
> What is wrong with the code beneath the comment?  
> What is the problem with include paths?  
> Why doesn't the cited code constitute a long-term solution?
> 

Well, in eight months, no one has suggested any answers to these questions. So I propose to yank the comment (config/gen/makefiles/dynpmc_pl.in line 60) and resolve the ticket unless I hear otherwise in seven days.

Thank you very much.
kid51

[jkeenan]

Replying to jkeenan:

Well, in eight months, no one has suggested any answers to these questions. So I propose to yank the comment (config/gen/makefiles/dynpmc_pl.in line 60) and resolve the ticket unless I hear otherwise in seven days.

No objections in the allocated time, so I yanked the comment in r43249.

Closing ticket. Thank you very much.

kid51