By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1608 - Build issue: k9copy uses internal dvdread headers, instead of system-provided ones
Summary: Build issue: k9copy uses internal dvdread headers, instead of system-provided...
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: non-core programs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: All Linux
: P5 normal
Assignee: Francois Andriot
URL:
Depends on:
Blocks:
 
Reported: 2013-08-02 04:42 CDT by Francois Andriot
Modified: 2013-08-13 17:45 CDT (History)
4 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
k9copy : use system-provided dvdread headers (4.64 KB, patch)
2013-08-02 04:44 CDT, Francois Andriot 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Francois Andriot 2013-08-02 04:42:55 CDT 
As the title says, k9copy comes with a "dvdread" folder which contains headers, which may be incompatible with those installed in the distribution.
Comment 1 Francois Andriot 2013-08-02 04:44:40 CDT 
Created attachment 1437 [details]
k9copy : use system-provided dvdread headers

The attached patch is an attempt to remove the included "dvdread" folder.
Apply the patch, remove the folder from sources (rm -rf ./dvdread) , then build k9copy.
Comment 2 Darrell 2013-08-04 23:46:22 CDT 
Perhaps I misunderstand, but what happens when dvdread is not installed from a distro package? Should the k9copy version then be used to compile k9copy?
Comment 3 Francois Andriot 2013-08-05 06:03:11 CDT 
First, if your distribution lacks the dvdread library, k9copy will miss the dvdread-related features, even with the internal headers.

What I'm afraid of, when building with internal header, is that they may be obsolete or incompatible with the actual dvdread library intalled on my system.

I do not see the purpose of having internal headers:
- Either you don't have/want dvdread, and you should have a "--disable-dvdread" option at build time
- Either you have and want it, and you install the "dvdread-devel" from your distribution to build k9copy .
Comment 4 Darrell 2013-08-05 10:59:26 CDT 
I suspect dvdread is standard on just about all distros. I was thinking that k9copy installed its own antiquated version when the distro version was not found
Comment 5 Timothy Pearson 2013-08-13 17:44:48 CDT 
Comment on attachment 1437 [details]
k9copy : use system-provided dvdread headers

Pushed to GIT in hash 71c6fa0.

Thanks for reporting and for the patch!