#234: mk_gitignore.pl (lib/Parrot/Manifest.pm) needs svk
---------------------+------------------------------------------------------
 Reporter:  fperrad  |       Owner:     
     Type:  bug      |      Status:  new
 Priority:  normal   |   Milestone:     
Component:  none     |     Version:     
 Severity:  medium   |    Keywords:     
     Lang:           |       Patch:     
 Platform:           |  
---------------------+------------------------------------------------------

Comment(by jkeenan):

 I recall that we encountered problems like this one or two years ago when
 SVK was the shiny new thing in VCSes, a role in which git has since
 usurped it.  It's also not surprising given that when
 ''tools/dev/mk_gitignore.pl'' was added to the repository, no unit tests
 were added to those run by ''make manifest_tests''.  (See attached
 coverage report for ''make manifest_tests''.

 Examining the code in ''lib/Parrot/Manifest.pm'', I believe the line
 causing the problem is:
 {{{
         cmd        => -d '.svn' ? 'svn' : 'svk',
 }}}
 So, if you don't have a ''.svn'' hidden directory underneath the top level
 of your sandbox, you're assumed to be running SVK instead of Subversion.
 My hunch is that you don't have such a ''.svn'' hidden directory in that
 location and, if you're running git, don't need one.  But you don't have
 SVK installed either; hence the failures.

 I'm not running git, so I don't know what an appropriate revision to
 ''lib/Parrot/Manifest.pm'' and the tests in ''t/manifest/*.t'' would be.
 François, could you, Barney or one of the other git advocates develop a
 patch?  I would be happy to work with you toward getting this working for
 you.

 Thank you very much[[BR]]
 kid51

-- 
Ticket URL: <https://trac.parrot.org/parrot/ticket/234#comment:1>
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