Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Request: support enumeration and traversal of submodules #210

Closed
cxw42 opened this issue Aug 5, 2019 · 6 comments
Closed

Request: support enumeration and traversal of submodules #210

cxw42 opened this issue Aug 5, 2019 · 6 comments

Comments

@cxw42
Copy link

cxw42 commented Aug 5, 2019

Thank you for this module, which I am using in a forthcoming project. I would like to be able to enumerate and traverse submodules. Would you be willing to add support for at least git_submodule_foreach and git_submodule_open? Thank you for considering this request!

(Related to #192, but reading rather than creating.)

@jacquesg
Copy link
Owner

jacquesg commented Aug 6, 2019

Let me see what I can do.

@jacquesg
Copy link
Owner

jacquesg commented Aug 6, 2019

https://github.com/jacquesg/p5-Git-Raw/tree/submodule has a work-in-progress version, could you test and see if it works for you?

cxw42 pushed a commit to cxw42/git-find that referenced this issue Aug 6, 2019
Requires development version of Git::Raw - see
jacquesg/p5-Git-Raw#210 (comment)

Invoke as:

    perl -I"$HOME/proj/git-find/lib" \
        -I"${HOME}/proj/p5-Git-Raw/Git-Raw-0.84/blib/arch" \
        -I"${HOME}/proj/p5-Git-Raw/Git-Raw-0.84/blib/lib" \
        "$HOME/proj/git-find/script/git-find" [args]
@cxw42
Copy link
Author

cxw42 commented Aug 6, 2019

Thank you for the quick response! It looks good in my initial tests :) .

@jacquesg
Copy link
Owner

Fixed by #212

@cxw42
Copy link
Author

cxw42 commented Aug 19, 2019

@jacquesg Would you be willing to release a CPAN version with this change? It is working well for me and I would like to be able to specify it as a regular dependency so my project can be installed via cpanm. Thanks!

@jacquesg
Copy link
Owner

Version 0.84 should be on CPAN shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants