Opened 13 years ago

Closed 13 years ago

Last modified 5 years ago

#7297 closed enhancement (fixed)

spkg's for libogg and libtheora

Reported by: whuss Owned by: whuss
Priority: major Milestone: sage-4.4.4
Component: packages: optional Keywords: video, animation
Cc: Merged in:
Authors: Wilfried Huss Reviewers: Pablo Angulo
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges


Packages for libogg and libtheora. The libtheora spkg installs the command line tool "png2theora" which can be used to encode a series of PNG images into a Theora video.

Change History (13)

comment:1 Changed 13 years ago by whuss

Status: newneeds_review

comment:2 Changed 13 years ago by mhampton

The end of my install for libtheora looks like this:

/bin/sh ./mkinstalldirs /Users/mh/sagestuff/sage-4.2/local/lib/pkgconfig
 /usr/bin/install -c -m 644 theora.pc /Users/mh/sagestuff/sage-4.2/local/lib/pkgconfig/theora.pc
 /usr/bin/install -c -m 644 theoradec.pc /Users/mh/sagestuff/sage-4.2/local/lib/pkgconfig/theoradec.pc
 /usr/bin/install -c -m 644 theoraenc.pc /Users/mh/sagestuff/sage-4.2/local/lib/pkgconfig/theoraenc.pc
cp: examples/.libs/png2theora: No such file or directory

real	0m32.161s
user	0m19.908s
sys	0m8.520s
sage: An error occurred while installing libtheora-1.1.1

Seems like things compiled OK though. This is on an intel mac, 10.5.


comment:3 Changed 13 years ago by mhampton

As far as I can tell, there is no attempt to actually build png2theora, its not a failure. But I am not sure how to edit the makefile to force this build.

comment:4 Changed 13 years ago by whuss

Is there the line

Build example code: ......... yes

at the end of configure?

Did it find libpng?

The option --enable-examples for configure should force the building of the examples.

comment:5 Changed 13 years ago by mhampton

I do see the "Build example code: ........... yes" string at the end of the configure output.

I don't see any indication of it looking for libpng, either a failure or success.

The only things that are being built inside of spkg/build/libtheora-1.1.1/src/examples are dump_video and dump_psnr.


comment:6 Changed 13 years ago by mhampton

I think the problem might be that I installed libogg, but this is not being detected by the script "newest_version", which looks in the standard directory for the spkg, which is not copied over.

comment:7 Changed 13 years ago by mhampton

OK, I tried copying the libogg spkg into the spkg/standard directory, but then the configure script fails with:

checking for Ogg... no * Could not run Ogg test program, checking why...

-I'm not sure what to try now.

comment:8 Changed 13 years ago by drkirkby

Report Upstream: N/A

Whats the point of

unset RM

in the spkg-install of libogg-1.1.4 ?

I'd either remove the line, or add a comment why it is needed.

comment:9 Changed 13 years ago by drkirkby

I would add 'set -e' before the 'cp' command in the spkg-install of libtheora-1.1.1. Then, if the copy fails, the spkg-install script will exit with a code of 1. Otherwise, this will appear to have installed correctly, even if the copy fails.

set -e 
cp examples/.libs/png2theora $SAGE_LOCAL/bin

comment:10 Changed 13 years ago by pang

Reviewers: pang
Status: needs_reviewpositive_review

Everything looks fine to me. I also don't understand the "unset RM" line, but I don't think it can hurt in any way. Not an expert in the reviewing process, but I'm giving positive review.

comment:11 Changed 13 years ago by mhansen

Authors: Wilfried Huss
Resolution: fixed
Reviewers: pangPablo Angula
Status: positive_reviewclosed

comment:12 Changed 13 years ago by mhansen

Reviewers: Pablo AngulaPablo Angulo

comment:13 Changed 5 years ago by dimpase

libtheora no longer builds; if it's not fixed, we'll have to remove it from optional packages...

Note: See TracTickets for help on using tickets.