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

use POSIX command . instead of "source" command #665

Merged
merged 3 commits into from Feb 27, 2021

Conversation

yamori813
Copy link
Contributor

I build OW on FreeBSD/i386. I have some error. This is fixed patch.

@jmalak
Copy link
Member

jmalak commented Feb 27, 2021

Thanks for your contribution.
Shell script change, it is OK.
Please remove the next change it is not acceptable.
If wmake build is not working then it must be fixed not to use gmake or is it bug in FreeBSD?
Generally POSIX solution is used if possible.
Please open issue for wmake build problem, it is most important part of build system that POSIX compatibility is required on *NIX systems.

@yamori813
Copy link
Contributor Author

Thanks comment.

I make original build.sh.

@jmalak
Copy link
Member

jmalak commented Feb 27, 2021

Please add fix for clean.sh together with build.sh.

@jmalak jmalak changed the title FreeBSD patch use POSIX command . instead of "source" command Feb 27, 2021
@jmalak jmalak merged commit 4ff5db4 into open-watcom:master Feb 27, 2021
@jmalak
Copy link
Member

jmalak commented Feb 27, 2021

clean.sh change were submitted.

@yamori813
Copy link
Contributor Author

Thanks.

I put new issue.

#666

@iainnicol
Copy link

Hi @yamori813. こんにちは、日本語に訳すのが下手で申し訳ありません。現在、Open Watcomは奇妙なソフトウェアライセンスを使用しています。私たちはOpen Watcomを「Apache 2.0 license plus LLVM Exceptions」で再ライセンスしたいと考えています。これで良いのでしょうか?詳しくはwikiをご覧ください。

Hi. Sorry for the bad translation into Japanese. Currently Open Watcom uses a weird software license. We want to relicense Open Watcom under “Apache 2.0 license plus LLVM Exceptions”. Is this okay with you? Please see the wiki for more information.

@yamori813
Copy link
Contributor Author

Sorry too late reply.

I think good that is change license.

@iainnicol
Copy link

Thank you :-)

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.

None yet

3 participants