#586: tools/dev/install_files.pl:  bug masked by missing file in
MANIFEST.generated
---------------------+------------------------------------------------------
 Reporter:  jkeenan  |       Owner:  jkeenan 
     Type:  bug      |      Status:  assigned
 Priority:  normal   |   Milestone:          
Component:  none     |     Version:  trunk   
 Severity:  medium   |    Keywords:          
     Lang:           |       Patch:  new     
 Platform:           |  
---------------------+------------------------------------------------------

Comment(by jkeenan):

 Replying to [comment:5 jkeenan]:
 > Replying to [comment:4 fperrad]:
 > >
 > > As I see, pirc don't need a config.
 >
 > Follow-up question:  How is 'pirc' currently built?
 >

 I now see that it is built via 'make pirc'.  Since it is not (currently)
 built by 'make', I don't believe it should be in 'MANIFEST.generated'.

 In r38446, I:

 -- corrected the faulty logic described in my OP to this TT;[[BR]]
 -- replaced `$_` with named lexicals in many locations;[[BR]]
 -- removed `pirc` and `pirc.exe` from ''MANIFEST.generated'';[[BR]]
 -- removed one comment which referred to a ticket which was earlier
 rejected;[[BR]]
 -- moved the POD to the end of the file so that you don't have to scroll
 down 5 screens to get to the code.

 ''tools/dev/install_files.pl'' now has a few of its problems corrected and
 conducts a successful installation.

 Thank you very much.[[BR]]
 kid51

-- 
Ticket URL: <https://trac.parrot.org/parrot/ticket/586#comment:6>
Parrot <https://trac.parrot.org/parrot/>
Parrot Development
_______________________________________________
parrot-tickets mailing list
[email protected]
http://lists.parrot.org/mailman/listinfo/parrot-tickets

Reply via email to