Page MenuHomeFreeBSD
Differential D20823

multimedia/oggvideotools: fix build with GCC-based architectures
ClosedPublic
Actions

Authored by pkubaj on Jul 1 2019, 11:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 27, 1:40 PM
Unknown Object (File)
Sun, Nov 23, 11:40 AM
Unknown Object (File)
Fri, Nov 21, 3:50 PM
Unknown Object (File)
Wed, Nov 19, 8:39 AM
Unknown Object (File)
Tue, Nov 18, 1:06 AM
Unknown Object (File)
Mon, Nov 17, 2:44 AM
Unknown Object (File)
Sun, Nov 16, 11:18 AM
Unknown Object (File)
Sun, Nov 16, 9:06 AM
View All 83 Files
Subscribers
None

Details

Reviewers
linimon
mat
tcberner
Commits
rP505953: multimedia/oggvideotools: fix build with GCC-based architectures
Summary

decoderTest.cpp needs include <cstring> for memset().

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 25151
Build 23845: arc lint + arc unit

Event Timeline

pkubaj created this revision.Jul 1 2019, 11:56 AM
Harbormaster completed remote builds in B25151: Diff 59257.Jul 1 2019, 11:56 AM
mat added inline comments.Jul 1 2019, 7:24 PM
multimedia/oggvideotools/files/patch-src_base_test_decoderTest.cpp
15–17

this hunk is not needed.

pkubaj marked an inline comment as done.Jul 1 2019, 8:25 PM
pkubaj added inline comments.
multimedia/oggvideotools/files/patch-src_base_test_decoderTest.cpp
15–17

Hm, strange. I removed this hunk from the patch and it's clearly not in the file, but Phabricator somehow adds it.

pkubaj updated this revision to Diff 59308.Jul 2 2019, 8:05 AM

Remove unnecessary hunk.

Harbormaster completed remote builds in B25178: Diff 59308.Jul 2 2019, 8:05 AM
pkubaj added inline comments.Jul 2 2019, 8:06 AM
multimedia/oggvideotools/files/patch-src_base_test_decoderTest.cpp
15–17

It's ok now, I accidentally submitted a new review instead of updating this one.

mat accepted this revision.Jul 5 2019, 4:49 PM
This revision is now accepted and ready to land.Jul 5 2019, 4:49 PM
Closed by commit rP505953: multimedia/oggvideotools: fix build with GCC-based architectures (authored by pkubaj). · Explain WhyJul 6 2019, 1:15 AM
This revision was automatically updated to reflect the committed changes.
pkubaj added a commit: rP505953: multimedia/oggvideotools: fix build with GCC-based architectures.

Revision Contents
Changeset List

PathSize
multimedia/
oggvideotools/
2 lines
files/
17 lines

Diff 59257

View Options

multimedia/oggvideotools/Makefile

Loading...
View Options

multimedia/oggvideotools/files/patch-src_base_test_decoderTest.cpp

Loading...