#4427 closed enhancement (fixed)
libvpx-v1.3.0
Reported by: | Fernando de Oliveira | Owned by: | Fernando de Oliveira |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | BOOK | Version: | SVN |
Severity: | normal | Keywords: | |
Cc: |
Description ¶
Change History (8)
comment:1 by , 11 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 11 years ago
comment:3 by , 11 years ago
Build and install fine.
It is working fine with ffmpeg, then, tested that ffmpeg builds without problem.
Shall I update the book?
comment:4 by , 11 years ago
BTW, I did not use
sed -i 's/cp -p/cp/' build/make/Makefile && chmod -v 644 vpx/*.h &&
and get:
$ ls -l /usr/include/vpx total 128 -rw-rw-r-- 1 fernando fernando 4045 Dez 8 08:30 svc_context.h -rw-rw-r-- 1 fernando fernando 12005 Dez 8 08:30 vp8cx.h -rw-rw-r-- 1 fernando fernando 3363 Dez 8 08:30 vp8dx.h -rw-rw-r-- 1 fernando fernando 5108 Dez 8 08:30 vp8.h -rw-rw-r-- 1 fernando fernando 19592 Dez 8 08:30 vpx_codec.h -rw-rw-r-- 1 root root 633 Mai 8 2012 vpx_codec_impl_bottom.h -rw-rw-r-- 1 root root 641 Mai 8 2012 vpx_codec_impl_top.h -rw-rw-r-- 1 fernando fernando 14106 Dez 8 08:30 vpx_decoder.h -rw-rw-r-- 1 fernando fernando 36933 Dez 8 08:30 vpx_encoder.h -rw-rw-r-- 1 fernando fernando 10712 Dez 8 08:30 vpx_image.h
This was uncompressed, built and installed as root. Have checked with an all unprivileged build/DEST-install, only difference is that the two "root:root" are "fernando:fernando".
So, chown is necessary, but I ma not sure about the other two instructions. Going to check Makefile.
Yes:
$ grep 'cp -p' build/make/Makefile $(qexec)cp -p $$< $$@
Thus, the sed seems necessary too. But do we need to change 664 into 644 in the installed files?
comment:5 by , 11 years ago
OK. Just the sed corrected everything, but I get 664, not 644 for the headers under /usr/include/vpx:
$ ls -l /usr/include/vpx total 120 -rw-rw-r-- 1 root root 4045 Dez 8 12:17 svc_context.h -rw-rw-r-- 1 root root 12005 Dez 8 12:17 vp8cx.h -rw-rw-r-- 1 root root 3363 Dez 8 12:17 vp8dx.h -rw-rw-r-- 1 root root 5108 Dez 8 12:17 vp8.h -rw-rw-r-- 1 root root 19592 Dez 8 12:17 vpx_codec.h -rw-rw-r-- 1 root root 14106 Dez 8 12:17 vpx_decoder.h -rw-rw-r-- 1 root root 36933 Dez 8 12:17 vpx_encoder.h -rw-rw-r-- 1 root root 10712 Dez 8 12:17 vpx_image.h -rw-rw-r-- 1 root root 1380 Dez 8 12:17 vpx_integer.h
comment:6 by , 11 years ago
Hmm, strange that a tarball has not been released yet. Maybe this is a pre-release tag. I can see that at least Arch has added the package, so I guess we could update it too.
The permissions for the header files should definitely be changed to 644.
comment:7 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
OK. Thanks, Ragnar.
I only mentioned the doubt about permissions, because in the thread, the discussion was that the headers had the permission 0640, and it seemed to me that the last "0" was the problem.
Fixed at r12358.
This seems to be considered a release, but only available using git:
I have produced libvpx-v1.3.0.tar.xz with directory name convention as in the book (tarball in the book is bz2), after removing .git .gitattributes .gitignore.
Have made a DESTDIR build and am now going to install and test.
Please, comments and/or objections are welcome.