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

Add option -v to expunge.ml #4830

Closed
vicuna opened this issue Jun 29, 2009 · 2 comments

Comments

Projects
None yet
2 participants
@vicuna
Copy link

commented Jun 29, 2009

Original bug ID: 4830
Reporter: mehdi
Assigned to: @alainfrisch
Status: closed (set by @xavierleroy on 2013-08-31T10:46:05Z)
Resolution: fixed
Priority: normal
Severity: feature
Category: ~DO NOT USE (was: OCaml general)
Monitored by: @glondu

Bug description

It would be nice (and useful feature) if expunge had a "-v" option to let the user enter modules to filter instead of modules to keep.

Please find attached a patch that adds this feature.
(The patch's author is Alain Frisch and it stolen from OCamlDuce's source code)

File attachments

@vicuna

This comment has been minimized.

Copy link
Author

commented Jan 20, 2012

Comment author: mehdi

Does "acknowledged" mean "wait forever without a reply"? (especially for such trivial change).

@vicuna

This comment has been minimized.

Copy link
Author

commented Jan 20, 2012

Comment author: @alainfrisch

Acknowledged means that no decision has been taken yet. Please understand that our resources are limited and that we focus on (what we consider) higher priorities.

One way to get a better treatment for feature requests is to motivate them. "It would be nice" is not a strong incentive to spend a few minutes on improving an undocumented tool.

I've integrated your patch (rev 12061).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.