Bug 36769 - New port: graphics/xvid
Summary: New port: graphics/xvid
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Mario Sergio Fujikawa Ferreira
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-04-05 00:10 UTC by Michael Nottebrock
Modified: 2002-04-09 00:40 UTC (History)
0 users

See Also:


Attachments
file.shar (3.12 KB, text/plain)
2002-04-05 00:10 UTC, Michael Nottebrock
no flags Details
patch-xvid (5.34 KB, text/plain; charset=us-ascii)
2002-04-06 20:39 UTC, Mario Sergio Fujikawa Ferreira
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Nottebrock 2002-04-05 00:10:01 UTC
xvid is an opensource videocodec, based on opendivx.
Comment 1 Mario Sergio Fujikawa Ferreira freebsd_committer freebsd_triage 2002-04-06 19:36:44 UTC
Responsible Changed
From-To: freebsd-ports->lioux

I'll handle this
Comment 2 Mario Sergio Fujikawa Ferreira freebsd_committer freebsd_triage 2002-04-06 20:39:30 UTC
Hi,

	I did a few changes to your port:

	1) Updated to snapshot 20020404

	2) Changed PORTVERSION to 0.20020404. The pre-fixing 0 is
a defensive measure. PORTVERSION has to be a increasing number
across updates but developer has been changing version numbering
systems. If he assigns a decreasing number, we can bump the 0 to
1;

	3) Added an autoconf trick. Check
files/{config.h.in,configure.in}. Unfortunaly, -STABLE does not
have stdint.h but it has inttypes.h. configure uses the one
it detects. This should be sent back to the developer;

	4) Added option WITH_OPTIMIZED_CFLAGS... optimizations
are known to break the ALPHA machines. Also, ports should not tweak
CFLAGS unless specified by the user. They should be optional;

	5) Make the port CC/CFLAGS safe (check both patch-Makefile
and post-patch target)

	6) Added ONLY_FOR_ARCHS=i386 since it has -DARCH_X86.
Does it work under other architectures? ALPHA? SPARC?

	7) Changed INSTALL to INSTALL_DATA

	8) Sorted pkg-plist

	I hope you agree with these changes. Some are suggestions,
some are port style changes... let me know what do you think, we
can discuss it. After all, this is your port.
	A few questions, is this a i386 only port? How did you
build it? Under -CURRENT? Is there stdint.h under -CURRENT? I
wouldn't know since I run -STABLE (which did not build this without
the autoconf hack).
	Please, send the autoconf patch back to the developer since
it helps getting this built under *BSD.

	Thanks for the nice port. We will need xvid bad now that opendivx seems dead and DivX 5 does not seem like coming to *BSD.

	Regards,


-- 
Mario S F Ferreira - DF - Brazil - "I guess this is a signature."
Computer Science Undergraduate | FreeBSD Committer | CS Developer
flames to beloved devnull@someotherworldbeloworabove.org
feature, n: a documented bug | bug, n: an undocumented feature
Comment 3 Mario Sergio Fujikawa Ferreira freebsd_committer freebsd_triage 2002-04-09 00:39:42 UTC
State Changed
From-To: open->closed

Committed with some style changes, thanks!