Skip to content

remove systype and attendant platform detection. #34

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

Merged
merged 1 commit into from
Aug 25, 2019

Conversation

alanpost
Copy link
Contributor

It's used in one place, make-makelib, to exclude ranlib on
platforms that no longer exist.

@alanpost alanpost added this to the 1.08 milestone Jul 19, 2019
@DerDakon
Copy link
Member

It would be cool if someone could tests this on a recent hpux system, all others are dead.

@alanpost alanpost force-pushed the pr-XX-remove-systype branch from dc3adee to 8003d85 Compare July 19, 2019 20:59
@alanpost alanpost added the build label Jul 19, 2019
@alanpost alanpost force-pushed the pr-XX-remove-systype branch from 8003d85 to 0972f20 Compare July 20, 2019 14:44
@alanpost alanpost force-pushed the pr-XX-remove-systype branch 2 times, most recently from a45431b to 32982e3 Compare August 6, 2019 14:03
@schmonz
Copy link
Member

schmonz commented Aug 14, 2019

It would be cool if someone could tests this on a recent hpux system, all others are dead.

According to this HP-UX 11i ranlib(1) manpage, it has ranlib these days.

If it turns out we're wrong and someone's build fails for lack of ranlib, then either they can work around it easily (and we can add a $PATH check for it instead of an OS check), or we'll need their help understanding how to link libraries on their system.

@schmonz schmonz modified the milestones: 1.08, 1.07 Aug 14, 2019
@schmonz schmonz requested a review from DerDakon August 14, 2019 03:01
@alanpost alanpost force-pushed the pr-XX-remove-systype branch from 32982e3 to d17f38d Compare August 14, 2019 03:02
@alanpost alanpost modified the milestones: 1.07, 1.08 Aug 14, 2019
It's used in one place, make-makelib, to exclude ranlib on
platforms that no longer exist.
@alanpost alanpost force-pushed the pr-XX-remove-systype branch from d17f38d to 97c4315 Compare August 25, 2019 14:30
@DerDakon DerDakon merged commit eed6095 into notqmail:master Aug 25, 2019
@alanpost alanpost deleted the pr-XX-remove-systype branch August 26, 2019 13:31
@DerDakon DerDakon modified the milestones: 1.09, 1.08 Dec 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants