Skip to content

Conversation

14NGiestas
Copy link
Owner

@14NGiestas 14NGiestas commented Jul 4, 2022

No description provided.

jvdp1 and others added 24 commits June 20, 2022 22:58
Co-authored-by: Ian Giestas Pauli <iangiestaspauli@gmail.com>
Co-authored-by: Ian Giestas Pauli <iangiestaspauli@gmail.com>
Co-authored-by: Ian Giestas Pauli <iangiestaspauli@gmail.com>
Co-authored-by: Ian Giestas Pauli <iangiestaspauli@gmail.com>
Co-authored-by: Ian Giestas Pauli <iangiestaspauli@gmail.com>
Co-authored-by: Ian Giestas Pauli <iangiestaspauli@gmail.com>
Co-authored-by: Ian Giestas Pauli <iangiestaspauli@gmail.com>
Co-authored-by: Ian Giestas Pauli <iangiestaspauli@gmail.com>
@14NGiestas
Copy link
Owner Author

Ok, @jvdp1 I've created this PR on my own fork of stdlib just to see if that pull/665 would fix the entire PR pull/662 so we don't need to wait as much.
The "fpm CI" it's satisfied but intel compilers don't allow the formatted IO using the '(A)' (character) descriptor. I guess either changing it to '(DT)' descriptor or just "importing" the write(unformatted) and then printing using print*, will do the trick.

I'll go ahead and push a few commits in your extract_demo branch then I will re-run the CI to see if the problem is gone or if there's anything else to fix.

You can see the intel CI here

The following tests FAILED:
	232 - padl (Failed)
	233 - padr (Failed)
	234 - replace_all (Failed)

@14NGiestas
Copy link
Owner Author

Great, now the example with padr passes which means using DT instead of A makes intel Fortran happy

Errors while running CTest
The following tests FAILED:
	232 - padl (Failed)
	234 - replace_all (Failed)

@14NGiestas 14NGiestas closed this Jul 4, 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.

2 participants