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

plusplus: Fix dependency file creation (multiple target patterns) #1073

Merged
merged 1 commit into from
Apr 27, 2023
Merged

Conversation

winspool
Copy link
Contributor

@winspool winspool commented Apr 26, 2023

Without the fix, OpenWatcom creates broken dependency files.
(everything is in one line)

GNU Make can build the c++ autotools project once (target files are missing),
but any following make to update the targets fails with a message similar to:

*** multiple target patterns. Stop

An example autotools project is attached:

./bootstrap
CC=owcc  CXX=owcc  CFLAGS="-v"  CXXFLAGS="-v"  ./configure
# first make works:
make

# a touch is optional:
# touch foo2.cpp
 
# a following make fails:
make

tst_foo.tar.gz

--
Regards Detlef

Without the fix, OpenWatcom creates broken dependency files.
GNU Make can build the c++ autotools project once,
but any following make fails with a message similar to:

*** multiple target patterns.  Stop

--
Regards Detlef
@jmalak
Copy link
Member

jmalak commented Apr 27, 2023

Thanks for your contribution

@jmalak jmalak merged commit 5040430 into open-watcom:master Apr 27, 2023
0 of 2 checks passed
@iainnicol
Copy link

Hi @winspool Detlef. 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.

@winspool
Copy link
Contributor Author

We want to relicense Open Watcom under “Apache 2.0 license plus LLVM Exceptions”. Is this okay with you?

Yes, sure

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