Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at
Bug 169082 - Review Request: physfs: Library to provide abstract access to various archives
Summary: Review Request: physfs: Library to provide abstract access to various archives
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Adrian Reber
QA Contact: David Lawrence
Depends On:
Blocks: FE-ACCEPT 169085
TreeView+ depends on / blocked
Reported: 2005-09-22 20:42 UTC by Tom "spot" Callaway
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2005-09-25 21:27:14 UTC
Type: ---

Attachments (Terms of Use)

Description Tom "spot" Callaway 2005-09-22 20:42:05 UTC
Spec Name or Url:
SRPM Name or Url:

PhysicsFS is a library to provide abstract access to various archives. It is
intended for use in video games, and the design was somewhat inspired by Quake
3's file subsystem. The programmer defines a "write directory" on the physical
filesystem. No file writing done through the PhysicsFS API can leave that
write directory, for security. For example, an embedded scripting language
cannot write outside of this path if it uses PhysFS for all of its I/O, which
means that untrusted scripts can run more safely. Symbolic links can be
disabled as well, for added safety. For file reading, the programmer lists
directories and archives that form a "search path". Once the search path is
defined, it becomes a single, transparent hierarchical filesystem. This makes
for easy access to ZIP files in the same way as you access a file directly on
the disk, and it makes it easy to ship a new archive that will override a
previous archive on a per-file basis. Finally, PhysicsFS gives you
platform-abstracted means to determine if CD-ROMs are available, the user's
home directory, where in the real filesystem your program is running, etc.

Comment 1 Adrian Reber 2005-09-23 08:52:03 UTC
* rpmlint is happy
* builds in mock
* clean installation and removal
* source matches upstream
* spec looks good
* correct usage of -devel
* sane scripts
* test binary works


If you BR doxygen, you could also generate the documentation and put it into the
 -devel package.

The INSTALL file could be removed.

I also think that it should be no problem that the header file from the -devel
package is directly in /usr/include as it is only one file and the name doesn't
sound too generic. If physfs will start to provide more include files in the
future a separate include directory might make more sense.

Note You need to log in before you can comment on or make changes to this bug.