Recent merge #55 breaks GCC builds

DIS(IEEE 1278.1) Fully Implemented in C++

Brought to you by: karljj1

This project can now be found here.

#107 Recent merge #55 breaks GCC builds

Milestone: 3.0.0

Status: closed-fixed

Owner: nobody

Labels: None

Priority: 5

Updated: 2022-02-13

Created: 2022-02-12

Creator: Gerard Lanois

Private: No

GCC is a popular compiler.

This merge #55 does not build with GCC. I tried both 9.3 and 11.2

https://sourceforge.net/p/kdis/code/merge-requests/55/

See attached log files.

2 Attachments

macosx_build_error.txt

ubuntu_build_error.txt

Discussion

Karl Jones - 2022-02-13

Sorry I accidentally deleted the post in the moderation queue. Thanks for sending the CI link, I have been meaning to get something like this setup. Ill look into the failure.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Karl Jones - 2022-02-13

I have pushed a fix. Im also working on setting up a github version of kdis which includes CI.
If all goes well ill move the project over.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.
- Gerard Lanois - 2022-02-13
  
  Thanks for fixing this. I tried it - works for me now. Let me know if you need any help with your CI rollout (eg, testing).
  
  If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
  - Anonymous
    
    Add attachments
    Cancel
    You seem to have CSS turned off. Please don't fill out this field.
    
    You seem to have CSS turned off. Please don't fill out this field.

Karl Jones - 2022-02-13

status: open --> closed-fixed

👍
1
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Anonymous