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

Script updates #32

Open
wants to merge 34 commits into
base: master
Choose a base branch
from
Open

Script updates #32

wants to merge 34 commits into from

Conversation

tilkinsc
Copy link
Owner

No description provided.

- Removed .exe hardcoded extensions, so scripts masking scripts can be
  run instead
- Default using curl instead of bitsadmin due to curl being native to
  windows now
- build.* build [lua_ver] has been changed to build.* driver [lua_ver]
  in the usage, due to the previous change going overlooked in the help
  message
@codecov-commenter
Copy link

codecov-commenter commented Oct 15, 2021

Codecov Report

Merging #32 (d44e139) into master (105d6fc) will not change coverage.
The diff coverage is n/a.

❗ Current head d44e139 differs from pull request most recent head 922aae5. Consider uploading reports for the commit 922aae5 to get more accurate results

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   17.24%   17.24%           
=======================================
  Files           5        5           
  Lines         603      603           
=======================================
  Hits          104      104           
  Misses        499      499           
Impacted Files Coverage Δ
src/luadriver.c 25.00% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tilkinsc
Copy link
Owner Author

 Qasim Abdullah (Travis CI)

Oct 15, 2021, 13:44 GMT-3
Hello Cody,

Thank you for reaching out! I will happily assist you.

While our Infra team is working on a deep analysis on this issue, following workaround has been suggested by them. Please, add it to before_install step and see if that resolves the issue:

sudo apt update && sudo apt install libgnutls-openssl27 libgnutls30

Thank you and we will follow-up on your reply.

Best,

--
Qasim

Your Friends @Travis CI

Test and Deploy with Confidence.
www.travis-ci.com

—
Travis CI GmbH, Rigaer Str.8, 10247 Berlin, Germany | GF/CEO: Randy Jacops | Contact: [email protected] | Amtsgericht Charlottenburg, Berlin, HRB 140133 B | Umsatzsteuer-ID gemäß §27 a Umsatzsteuergesetz: DE282002648

@tilkinsc
Copy link
Owner Author

I have to give up on travis-ci while they fix their issues regarding openssl.

- removed call per each batch file call in appveyor so everything is
  synchronized
- added tests to appveyor
- github ci removed calls also so everything is synchronized
- reorgnized mingw build script gcc arguments
- revert saved change where mingw build script added temporary debug
  information
- added a ci folder
- migrated all ci files to ci folder
- updated yml files to execute builds and test sin new ci folder
@tilkinsc
Copy link
Owner Author

Looks like there is a race error in building due to not being able to call my scripts and exit them with exit /b 0 back to the caller. community/community#6432

@tilkinsc
Copy link
Owner Author

Looks like circle ci has the same exact issue as github.

@tilkinsc
Copy link
Owner Author

This library depends too heavily on objcopy, which is used to take puc-lua and run files through the official compiler code instead of reimplementing the compiler. It would be proper to reimplement to drop this dependency.

tilkinsc and others added 2 commits December 11, 2022 08:54
- finally fixed locale code
- linux now builds correctly
- linux now installs
- makefile fixed
- added uninstaller to makefile and linux
- dll path is searched relative to executable
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