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

retire some support stuff #866

Merged
merged 2 commits into from
Jul 6, 2022
Merged

Conversation

ipelupessy
Copy link
Member

cleanup no longer used utilities from the support directory (the distribution support). It no longer worked (well, maybe its fixable) and should be done differently anyway

maybe also the distribute directory?

it probably doesn't work anymore, it was not 1005 correct and should be
in a seperate repo if revived
@ipelupessy
Copy link
Member Author

yea, distribute should retire too:

  • its not likely to be working right now, I don't anyone has time to fix it
  • the approach taken is not 100% correct, it worked, but but mostly by coincendence (it was supposed to contain all libs, but it didn't, just that most distributions tested had compatible libs
  • if revived I would advocate a seperate repo for this kind of tool

@rieder
Copy link
Member

rieder commented Jun 29, 2022

Agreed...

@ipelupessy
Copy link
Member Author

interestingly, the build script was leaking a password (luckily a long shutdown svn repo)

@ipelupessy ipelupessy merged commit a6884de into amusecode:main Jul 6, 2022
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.

None yet

2 participants