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

Unforking our ClamAVJ #9

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Conversation

gregjan
Copy link

@gregjan gregjan commented Aug 11, 2014

Hi Phil,
We made some changes to ClamAVJ for use in our digital repository. This PR shows what we did. We are interested in contributing the changes, if you would like them. If you would, then we could adjust the group-id and work with you on any other comments you have.

The main change was to support passing file paths to "SCAN", as opposed to streaming. This involved a little extension to ScanResult.

@gregjan
Copy link
Author

gregjan commented Aug 28, 2014

Thanks again for all your work on this, Phil. Just let us know if you are ever interested in these changes.

@philvarner
Copy link
Owner

Sorry I hadn't replied yet, thanks for prodding. Yes, I am -- I haven't
really done much work with the project for the last couple years, but I
recently got assigned to do some more with it at work, so I'm going to
revive it.

--Phil

On Thu, Aug 28, 2014 at 11:03 AM, Gregory Jansen [email protected]
wrote:

Thanks again for all your work on this, Phil. Just let us know if you are
ever interested in these changes.


Reply to this email directly or view it on GitHub
#9 (comment).

@philvarner
Copy link
Owner

Does anyone want to just take over ownership of this project? I've obviously not maintained it in a while.

@bbpennel
Copy link

bbpennel commented Nov 7, 2018

@philvarner Sorry, I just noticed this reply. That may be possible if you do not feel like you will have time. Do you have any concerns with the state of the project or with transferring ownership?

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

Successfully merging this pull request may close these issues.

5 participants