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

Update dependencies in build hook (main) #130

Closed
wants to merge 1 commit into from

Conversation

korydraughn
Copy link
Contributor

No description provided.

@trel
Copy link
Member

trel commented Oct 11, 2023

yes

@korydraughn
Copy link
Contributor Author

The following are referenced as dependencies in the build hook.

  • irods-externals-cpr1.3.0-1
  • irods-externals-elasticlientd68e30e3-0

We need to figure out what to do about #128.

@korydraughn
Copy link
Contributor Author

See comment at #128 (comment) for current plan.

@alanking
Copy link
Contributor

Do we need this anymore since #128 has been resolved?

@korydraughn
Copy link
Contributor Author

This isn't needed anymore.

Why isn't github reporting a conflict or that this is a no-op? Am I blind or something?

@alanking
Copy link
Contributor

Well, it is only a one-line change. Maybe it can rebase cleanly?

@korydraughn
Copy link
Contributor Author

Well in any case, this PR isn't needed.

I'm going to close it unless you have opinions.

@alanking
Copy link
Contributor

Yep, agreed. Close it

@korydraughn korydraughn deleted the build_hook.m branch February 23, 2024 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants