Skip to content
This repository has been archived by the owner on May 9, 2024. It is now read-only.

Protobuf patch has trailing whitespace triggering warning. #1

Closed
aregm opened this issue Dec 13, 2018 · 1 comment
Closed

Protobuf patch has trailing whitespace triggering warning. #1

aregm opened this issue Dec 13, 2018 · 1 comment
Assignees
@vnlitvinov
Labels
bug Something isn't working

Comments

@aregm
Copy link

aregm commented Dec 13, 2018

Applying patch in Windows:
aregm@Barmaley MINGW64 /c/Projects/pysamprof (master)
$ cd 3rd_party/protobuf-c && git apply ../protobuf-c-vs2008-support.patch
../protobuf-c-vs2008-support.patch:36: trailing whitespace.

../protobuf-c-vs2008-support.patch:39: trailing whitespace.

warning: protobuf-c/protobuf-c.h has type 100644, expected 100755
warning: protoc-c/c_message.cc has type 100644, expected 100755
warning: 2 lines add whitespace errors.
@aregm aregm added the bug Something isn't working label Dec 13, 2018
vnlitvinov pushed a commit that referenced this issue Dec 18, 2018
@anmyachev
fix warnings for protobuf-c-vs2008-support.patch; added lost header a…
02f760d 
…nd change _BSD_SOURCE -> _DEFAULT_SOURCE for successful build on Ubuntu 18.04 (#1)

* fix warnings for protobuf-c-vs2008-support.patch
* fix lost header, change _BSD_SOURCE -> _DEFAULT_SOURCE
@vnlitvinov
Copy link
Contributor

Should be fixed by 02f760d.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@vnlitvinov vnlitvinov

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aregm @vnlitvinov