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

Merge changes from NOIRLABs version #12

Merged
merged 7 commits into from
Feb 4, 2024
Merged

Merge changes from NOIRLABs version #12

merged 7 commits into from
Feb 4, 2024

Conversation

olebole
Copy link
Member

@olebole olebole commented Apr 17, 2023

This PR contains (most of) the changes made in https://github.com/noirlab-iraf/fitsutil

Not applied:

  • cfitsio updates (we use the cfitsio library provided by IRAF),
  • mkfloat.sh usage (we use mkfloat; it is better that the tool name does not reflect the used shell type)
  • Replacing realloc() calls with calloc() in noirlab-iraf/fitsutil@008b762 (see Fix slines handling in fgwrite.c #8)
  • Adding and removing binaries
  • Re-(ab)using macosx as architecture name for Mac/ARM: First, there is no advantage to re-use the name for a different architecture, and then, the name is wrong now: the current OS for Macs is called macOS and no longer Mac OS X. macos64 was introduced in IRAF 2.17 and will be kept now.

Current status is noirlab-iraf/fitsutil@ecc4f91

Cc @mjfitzpatrick who authored the changes.

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