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

Marked rebar_utils:find_source/3 as deprecated #2901

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

ariel-anieli
Copy link

@ariel-anieli ariel-anieli commented Jun 21, 2024

@ferd
Copy link
Collaborator

ferd commented Jul 2, 2024

This is a riskier change to merge in because people in plugins might be relying on rebar_utils to make things work. So we may want to keep the function there and exported, but mark it as deprecated for xref.

* introcuded by dec4846
* introcuded to workaround releases prior R20 not having filelib:find_source/3 [1]

[1] https://www.erlang.org/doc/apps/stdlib/filelib.html#find_source/3
@ariel-anieli ariel-anieli force-pushed the filelib_find_source branch from 1c5567c to bf27f7a Compare July 2, 2024 20:37
@ariel-anieli ariel-anieli changed the title Removed filelib_find_source macro, and rebar_utils:find_source/3 Marked rebar_utils:find_source/3 as deprecated Jul 2, 2024
@ariel-anieli
Copy link
Author

Here is it, @ferd; thanks for the heads up.

@ferd ferd merged commit 1ed6c2e into erlang:main Jul 3, 2024
6 checks passed
@ferd
Copy link
Collaborator

ferd commented Jul 3, 2024

Ah I merged quick. Do you think that given the argument types we could have replaced the implementation with the filelib one? So you know, we keep the entry point but no longer own the implementation.

@ariel-anieli
Copy link
Author

That's true, @ferd; give me some time, I am pushing out a new PR.

@ariel-anieli
Copy link
Author

@ferd, PR is #2904.

@ariel-anieli ariel-anieli deleted the filelib_find_source branch July 3, 2024 22:41
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.

2 participants