Jo-Philipp Wich [Mon, 13 Feb 2017 17:07:04 +0000 (18:07 +0100)]
libopkg: fix hex encoding/decoding, add checksum getter/setter
Fix broken hex encoding/decoding due to the use of signed chars and introduce
new convenience getters and setters to simplify the checksum usage.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Mon, 13 Feb 2017 17:07:04 +0000 (18:07 +0100)]
libopkg: store checksums in binary form, use integer index for architecture
Instead of storing a copy of the architecture string and architecture
priority value in each pkg_t instance, declare a 3 bit wide field which
allows referencing the architecture in the global config array by index.
The 3 bit field allows referencing up to 8 different architectures which
is more than enough for the systems we target with opkg. Another nice side
effect is that we can coalesce this field with other flag values in pkg_t,
saving 4 bytes for an int member.
Also convert the hexadecimal checksums to binary format before storing them in
pkg_t's blob buffer to save 50% of the space per checksum.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Mon, 13 Feb 2017 13:31:24 +0000 (14:31 +0100)]
pkg: move active_list structure out of pkg_t
The active list head is only used by a tiny fraction of the allocated
package structures in memory so do not waste heap memory by allocating
space for it in all loaded packages.
Instead allocate active list heads dynamically where needed and point
them to the corresponding packages.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Sun, 12 Feb 2017 20:43:11 +0000 (21:43 +0100)]
pkg: convert most other struct members into dynamic blob buffer fields
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 21:56:13 +0000 (22:56 +0100)]
pkg: store size, installed size and installed time info in blob buffer
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 16:14:53 +0000 (17:14 +0100)]
pkg: use a blob buffer in pkg_t to store variable fields
Adds infrastructure to use a blob buffer in pkg_t to store and retrieve
optional dynamic fields.
This reduces the size of the pkg_t structure itself and lays the
ground work for further optimization.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 15:21:14 +0000 (16:21 +0100)]
pkg: coalesce soem flag members into a packed bit field
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 10:20:23 +0000 (11:20 +0100)]
treewrite: use Lindent to reformat to kernel coding style
Use the kernels "Lindent" script to reformat files using kernel coding style.
This should make subsequent refactoring much easier.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 10:06:38 +0000 (11:06 +0100)]
libopkg: clarify messages and errors related to downloads
Clarify opkg's messages related to downloads:
* more visible error message for package list download failure
* separate error message for signature file download error
* if wget returns 4, signal the network error more clearly
* remove '.' from end of filenames and URLs
* try signature check only if the package list was downloaded ok.
Signed-off-by: Hannu Nyman <hannu.nyman@iki.fi>
Jo-Philipp Wich [Fri, 10 Feb 2017 10:05:44 +0000 (11:05 +0100)]
libopkg: run prerm scripts for the old version also on upgrade
To make sure we properly restart services on upgrade we need to
call the prerm script of the old package, in case the init script
changes (or vanishes).
Signed-off-by: Jonas Gorski <jonas.gorski@gmail.com>
Acked-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 10:04:10 +0000 (11:04 +0100)]
libopkg: fix use-after-free with duplicate packages on the command line
When the same package file is specified multiple times on the opkg install
command line, the name pointer on the argv array becomes stale after the
package structures have been merged, leading to invalid memory accesses
upon install.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:57:45 +0000 (10:57 +0100)]
cli: implement --size
Implement a new switch "--size" which causes opkg to print the package size
in the listing output.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:44:17 +0000 (10:44 +0100)]
cli: implement --lists-dir
Add a new flag --lists-dir which allows setting the package feed list
directory from command arguments.
This is required to be able to use opkg without any configuration file.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:43:57 +0000 (10:43 +0100)]
libopkg: fix --force-checksum to cover sha256sum cases as well
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:41:51 +0000 (10:41 +0100)]
libopkg: make MD5 support optional
Disable md5 support code by default and introduce a new configure flag
"--enable-md5" which allows reenabling the code.
This shrinks the opkg executable by a few KB.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:40:04 +0000 (10:40 +0100)]
libopkg: drop support for Release files
Remove support for Release files as this facility is unused in LEDE/OpenWrt.
Removing this allows us to shrink the opkg executable a little bit.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:37:02 +0000 (10:37 +0100)]
cli: implement --force-signature
Implement a new "--force-signature" flag which causes opkg to ignore signature
verification errors.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:34:21 +0000 (10:34 +0100)]
libopkg: add support for signature checking through usign
Adds a new configure switch "--enable-usign" which enables code to perform
package feed signature checking using an external "/usr/sbin/opkg-key" helper
program.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:31:55 +0000 (10:31 +0100)]
libopkg: pass PKG_UPGRADE variable to package scripts
When installing or upgrading packages, pass an PKG_UPGRADE environment
variable to maintainer scripts to allow the to perform different actions
depending on whether a package is freshly installed or upgraded.
During installs the PKG_UPGRADE variable will be set to 0 and during
upgrades to 1.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:30:33 +0000 (10:30 +0100)]
cli: implement --force-checksum
Introduce a new --force-checksum flag which allows installing packages even if
the checksum does not match. This is useful for development and debugging.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:29:44 +0000 (10:29 +0100)]
libopkg: suppress empty provides
When writing out status information, suppress the Provides field if there
are no providers.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:28:09 +0000 (10:28 +0100)]
libopkg: discard empty control file fields
If the resulting string is empty after whitespace trimming, free it and
store NULL instead of an empty string buffer.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:24:58 +0000 (10:24 +0100)]
libopkg: store compressed package lists
To save storage space, store the package lists in compressed form and
decompress them on-the-fly during parsing.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:19:14 +0000 (10:19 +0100)]
libbb: use external gzip command as I/O layer
Remove the builtin inflate code and rely on the external gzip command instead
to inflate data streams.
This change is needed to properly support vfork(). We cannot use the builtin
code because it uses a single global state which will get clobbered when
recursively inflating nested archives.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:18:18 +0000 (10:18 +0100)]
cli: implement find command
Introduce a new opkg "find" command which matches both the name and the
description of packages.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:16:33 +0000 (10:16 +0100)]
cli: implement --nocase flag
Introduce a new --nocase flag which causes all opkg pattern matching
operations to ignore the case.
This is useful to find packages with uppercase letters in their name.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:14:02 +0000 (10:14 +0100)]
libopkg: line-wrap descriptions only on interactive terminals
Various programs call opkg and extract information from its list and info
commands. To simplify parsing the output, do not line-wrap long description
lines if opkg is not talking to a tty.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:04:57 +0000 (10:04 +0100)]
libopkg: fix double-free crash on recursive package removal
The opkg_remove_dependent_pkgs() function will trigger a pkg_vec_free() and
break the loop when an error is returned by opkg_remove_pkg().
Since the vector is freed anyway after the loop, a double free occurs,
leading to a segmentation fault.
This situation commonly happens when recursively deleting packages in
conjunction with --autoremove where dependent packages might already have
been removed by previous iterations.
Simply break the loop without prematurely freeing the vector to fix this.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:03:19 +0000 (10:03 +0100)]
libopkg: do not rely on getline()
OpenWrt/LEDE uses opkg on both the target systems and the build hosts and not
every supported host system provides a getline() implementation.
Rewrite affected code to not require the getline() function.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 09:02:42 +0000 (10:02 +0100)]
libopkg: print error messages to stderr
Print the collected error list to stderr isntead of stdout. This is useful
for wrapping programs that treat stdout and stderr differently.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 08:59:59 +0000 (09:59 +0100)]
libopkg: strip trailing white space in conffile specification
The conffile entries in package control lines may have trailing white space,
causing opkg to fail looking up the file paths.
Trim white space to avoid this error.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 08:58:20 +0000 (09:58 +0100)]
cli: default to /etc/opkg.conf
If neither --conf-file nor --offline-root are specified, default to the
traditional /etc/opkg.conf location to retain compatibility with OpenWrt/LEDE.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 08:57:18 +0000 (09:57 +0100)]
cli: enforce argument to upgrade command
Blindly upgrading all upgradable packages usually brick the system, so disable
support for argument-less upgrade commands for now.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 08:54:34 +0000 (09:54 +0100)]
build: statically link libopkg
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 08:53:40 +0000 (09:53 +0100)]
build: allow overriding host cpu with make variable
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 08:51:44 +0000 (09:51 +0100)]
build: remove output shaving support
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Jo-Philipp Wich [Fri, 10 Feb 2017 08:50:20 +0000 (09:50 +0100)]
build: missing pkg.m4
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
google@wwsnet.net [Fri, 8 Apr 2011 13:33:11 +0000 (13:33 +0000)]
Fix compilation after r617
git-svn-id: http://opkg.googlecode.com/svn/trunk@618
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Fri, 8 Apr 2011 04:30:10 +0000 (04:30 +0000)]
Don't include the source URI in the cached filename.
This avoids multiple downloads in the case where a repository is simply a
mirror of another. The old, uri mangled, filename is still checked to ensure
backwards compatibility with existing caches.
Patch from robert.melchers@gmail.com.
git-svn-id: http://opkg.googlecode.com/svn/trunk@617
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
javiplx@gmail.com [Thu, 7 Apr 2011 16:11:55 +0000 (16:11 +0000)]
Read the contents of lists files coming from 'dist' entries
git-svn-id: http://opkg.googlecode.com/svn/trunk@616
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
javiplx@gmail.com [Thu, 7 Apr 2011 16:07:25 +0000 (16:07 +0000)]
Download all the valid Packages files for dist entries
git-svn-id: http://opkg.googlecode.com/svn/trunk@615
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
javiplx@gmail.com [Thu, 7 Apr 2011 15:53:24 +0000 (15:53 +0000)]
Introduce a new pkg_src list in global configuration to store the contents of 'dists' entries for apt-alike repositories
git-svn-id: http://opkg.googlecode.com/svn/trunk@614
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
javiplx@gmail.com [Thu, 7 Apr 2011 15:51:52 +0000 (15:51 +0000)]
Fix mistype in define clause
git-svn-id: http://opkg.googlecode.com/svn/trunk@613
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
javiplx@gmail.com [Thu, 7 Apr 2011 15:35:24 +0000 (15:35 +0000)]
Code to handle the apt-alike configuration entries
git-svn-id: http://opkg.googlecode.com/svn/trunk@612
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
javiplx@gmail.com [Thu, 7 Apr 2011 15:17:27 +0000 (15:17 +0000)]
Create parse_util with some of the non-public functions from pkg_parse.c
git-svn-id: http://opkg.googlecode.com/svn/trunk@611
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
javiplx@gmail.com [Thu, 7 Apr 2011 15:03:54 +0000 (15:03 +0000)]
Add flag in opkg_download calls to reduce the severity of the 'failed download' message
git-svn-id: http://opkg.googlecode.com/svn/trunk@610
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Tue, 22 Feb 2011 03:39:04 +0000 (03:39 +0000)]
Fix opkg_re_read_config_files().
This fixes segfaults when opkg_update_package_lists() is called multiple times.
git-svn-id: http://opkg.googlecode.com/svn/trunk@609
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Tue, 22 Feb 2011 03:38:50 +0000 (03:38 +0000)]
Fix memory leak when !defined(HAVE_GPGME) and !defined(HAVE_OPENSSL).
git-svn-id: http://opkg.googlecode.com/svn/trunk@608
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 21 Feb 2011 04:45:33 +0000 (04:45 +0000)]
Add regress test for the bug from r605.
git-svn-id: http://opkg.googlecode.com/svn/trunk@607
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 21 Feb 2011 04:24:55 +0000 (04:24 +0000)]
List some sub-commands that were missing from the --help output.
git-svn-id: http://opkg.googlecode.com/svn/trunk@606
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 21 Feb 2011 04:24:42 +0000 (04:24 +0000)]
Get the owner of the file_name after the offline root has been stripped.
Fixes a problem where the old package maintains that it owns a file even
though the new pkg has been installed with --force-overwrite.
git-svn-id: http://opkg.googlecode.com/svn/trunk@605
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Fri, 18 Feb 2011 00:02:14 +0000 (00:02 +0000)]
Allow vfork()ing an external gunzip binary instead of using fork().
Patch from Mike Westerhof, with minor modifications to allow the use of both
GNU gunzip and busybox gunzip. His original patch header follows.
This patch allows a user to set an environment variable to cause opkg to
select either the built-in gunzip code or an external gunzip utility, in
order to dodge the OOM Killer.
The built-in code is, of course, is the most desirable way to use opkg,
since it is far more efficient. However, the built-in code can trigger
the OOM (out of memory) killer on small-memory machines, like the 32MB
NSLU2. This occurs because a standard fork will duplicate the entire
address space of the parent. Since opkg reads the entire feed database
into memory, this problem is compounded by large feeds.
This patch introduces a means for the user to cause opkg to use vfork()
instead -- vfork() does not behave in the same manner as fork(), and
does not trigger the OOM killer. However, the semantics of vfork() are
such that it cannot run the built-in gunzip code. Instead, it must
exec() an external utility to perform the gunzip operation. It seems
counter-intuitive, but the vfork()/exec() approach is the only good way
to avoid triggering the dreaded OOM killer.
In order to use this, the user must manually set the OPKG_USE_VFORK
environment variable to any value. For example:
$ OPKG_USE_VFORK=1 opkg install samba
The external utility used to do the gunzip operation is "busybox gunzip".
It would have been nice to be able to just invoke "gunzip", but the
full gunzip executable behaves slightly differently than does busybox,
generating annoying warning messages.
This is an update of the original patch by Mike Westerhof, Dec 2008.
Mike Westerhof, Feb 2011
git-svn-id: http://opkg.googlecode.com/svn/trunk@604
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 14 Feb 2011 02:47:09 +0000 (02:47 +0000)]
Delete package sources that fail the signature check.
From Bernhard Guillon.
git-svn-id: http://opkg.googlecode.com/svn/trunk@603
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 14 Feb 2011 02:46:33 +0000 (02:46 +0000)]
Call gpgme_check_version() before using other gpgpe functions.
The function initializes some sub-systems, and for this reason alone
it must be invoked early in your program, before you make use of the
other functions in GPGME.
see http://pyme.sourceforge.net/doc/gpgme/Library-Version-Check.html
Patch from Bernhard Guillon.
git-svn-id: http://opkg.googlecode.com/svn/trunk@602
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 14 Feb 2011 02:46:01 +0000 (02:46 +0000)]
Add overlay_root config option. Opkg checks this location for available space.
This option is useful in the case where root is mounted ro, and another rw
filesystem is overlaid on top with e.g. mini_fo.
From Nicolas Thill via OpenWrt.
git-svn-id: http://opkg.googlecode.com/svn/trunk@601
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 14 Feb 2011 02:45:32 +0000 (02:45 +0000)]
Add missing space.
git-svn-id: http://opkg.googlecode.com/svn/trunk@600
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Wed, 2 Feb 2011 00:53:46 +0000 (00:53 +0000)]
Fix depmod intercept. Patch from Andreas Oberritter.
* If depmod -a ${KERNEL_VERSION} is called, the version should be retained.
* The script calls depmod -A, which does nothing if the installed module's
timestamp is older than modules.dep's timestamp. As modules generally
inherit the timestamp of the build, this would result in depmod doing nothing.
git-svn-id: http://opkg.googlecode.com/svn/trunk@599
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
pixdamix [Tue, 1 Feb 2011 14:36:50 +0000 (14:36 +0000)]
Fixed a bug causing segfaults when an upgraded package has less dependencies than the old package.
git-svn-id: http://opkg.googlecode.com/svn/trunk@598
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Thu, 23 Dec 2010 01:38:57 +0000 (01:38 +0000)]
Create the regress offline_root from scratch.
git-svn-id: http://opkg.googlecode.com/svn/trunk@596
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Thu, 23 Dec 2010 01:38:40 +0000 (01:38 +0000)]
Fix potential undefined references to FILE, for libopkg users.
Patch from Richard Purdie.
git-svn-id: http://opkg.googlecode.com/svn/trunk@595
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Thu, 23 Dec 2010 01:38:25 +0000 (01:38 +0000)]
Don't truncate long symlink paths.
Patch from Richard Purdie. Tracked as Issue #72. Original patch header follows.
If a tarball contains a long symlink (over 100 chars) in a longpath (over 100
chars) then the resulting link or path can be truncated to 100 chars.
This is due to a bug where if both 'L' and 'K' entries are found in the tarball,
only the first one takes affect due to get_header_tar recursively calling itself.
To fix this, process longname and linkname at the end of the function rather
than the start after any subcalls have taken place.
Richard Purdie
22/12/2010
git-svn-id: http://opkg.googlecode.com/svn/trunk@594
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Thu, 23 Dec 2010 01:37:58 +0000 (01:37 +0000)]
Add regress test for issue72.
git-svn-id: http://opkg.googlecode.com/svn/trunk@593
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Thu, 23 Dec 2010 01:37:42 +0000 (01:37 +0000)]
Don't gitignore the regress Makefile.
git-svn-id: http://opkg.googlecode.com/svn/trunk@592
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Thu, 23 Dec 2010 01:37:18 +0000 (01:37 +0000)]
Don't print the function name when just outputting a \n.
git-svn-id: http://opkg.googlecode.com/svn/trunk@591
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 13 Dec 2010 04:39:47 +0000 (04:39 +0000)]
Remove stray exit() in the regress test.
git-svn-id: http://opkg.googlecode.com/svn/trunk@590
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 13 Dec 2010 04:39:29 +0000 (04:39 +0000)]
Fix missing declarations, found by Clang.
git-svn-id: http://opkg.googlecode.com/svn/trunk@589
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 13 Dec 2010 04:39:11 +0000 (04:39 +0000)]
Fix if statement with empty body due to stray semicolon.
Found with Clang.
git-svn-id: http://opkg.googlecode.com/svn/trunk@588
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Wed, 24 Nov 2010 00:01:35 +0000 (00:01 +0000)]
Abort package removal if the prerm script of a package returns non zero.
This can be overridden with --force-remove.
Also, improve propagation of errors up the call stack for related errors.
From Sergey 'Jin' Bostandzhyan.
git-svn-id: http://opkg.googlecode.com/svn/trunk@587
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 22 Nov 2010 23:48:31 +0000 (23:48 +0000)]
Print the package name corresponding to a failed script.
From Sergey 'Jin' Bostandzhyan.
git-svn-id: http://opkg.googlecode.com/svn/trunk@586
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Fri, 19 Nov 2010 06:19:25 +0000 (06:19 +0000)]
Fix formatting issues found with the last commit.
git-svn-id: http://opkg.googlecode.com/svn/trunk@585
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Fri, 19 Nov 2010 06:19:11 +0000 (06:19 +0000)]
Use printf format specifier attribute for opkg_message().
git-svn-id: http://opkg.googlecode.com/svn/trunk@584
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Fri, 19 Nov 2010 04:30:47 +0000 (04:30 +0000)]
Remove an unnecessary allocation.
According to SUS3:
If the value of n is zero on a call to snprintf(), nothing shall be written, the number of bytes that would have been written had n been sufficiently large excluding the terminating null shall be returned, and s may be a null pointer.
git-svn-id: http://opkg.googlecode.com/svn/trunk@583
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Fri, 19 Nov 2010 03:56:06 +0000 (03:56 +0000)]
Check return codes from vsnprintf and vfprintf.
git-svn-id: http://opkg.googlecode.com/svn/trunk@582
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Fri, 19 Nov 2010 03:55:52 +0000 (03:55 +0000)]
Rewrite sprintf_alloc.
The example in printf(3) that this was taken from should not be used as
it ignores negative return codes from vsnprintf, instead allocating more
memory. Given an error from vsnprintf, this would loop until the process'
virtual memory is exhausted.
So we just exit in the event of a problem instead.
git-svn-id: http://opkg.googlecode.com/svn/trunk@581
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Fri, 19 Nov 2010 03:55:38 +0000 (03:55 +0000)]
Fix indentation.
git-svn-id: http://opkg.googlecode.com/svn/trunk@580
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 15 Nov 2010 22:47:28 +0000 (22:47 +0000)]
opkg_find_package() should not return garbage if it cannot find the package.
From juliosao via Issue #69.
git-svn-id: http://opkg.googlecode.com/svn/trunk@579
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 15 Nov 2010 22:47:14 +0000 (22:47 +0000)]
This is not a bug any longer.
It was fixed long ago, as is clear by the code in question.
git-svn-id: http://opkg.googlecode.com/svn/trunk@578
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Thu, 11 Nov 2010 00:23:29 +0000 (00:23 +0000)]
Remove an erroneous comment.
git-svn-id: http://opkg.googlecode.com/svn/trunk@577
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
google@wwsnet.net [Tue, 5 Oct 2010 07:21:00 +0000 (07:21 +0000)]
Implement "list-changed-conffiles" command - based on patch by Yann Lopez <yann.lopez@gmail.com>.
git-svn-id: http://opkg.googlecode.com/svn/trunk@576
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Tue, 21 Sep 2010 01:30:51 +0000 (01:30 +0000)]
Remove dead code, sprintf_alloc() cannot fail. Opkg will exit instead.
git-svn-id: http://opkg.googlecode.com/svn/trunk@574
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Tue, 21 Sep 2010 01:25:04 +0000 (01:25 +0000)]
A proper fix this time. From Sergey 'Jin' Bostandzhyan <jin@mediatomb.cc>.
git-svn-id: http://opkg.googlecode.com/svn/trunk@573
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Tue, 21 Sep 2010 01:24:51 +0000 (01:24 +0000)]
Revert "Don't call opkg_conf_deinit() if opkg_conf_init() fails."
This reverts commit
12e726ce863c221d32d20ba41242c2fcb798a909.
I should pay more attention. This does not fix the problem.
git-svn-id: http://opkg.googlecode.com/svn/trunk@572
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Tue, 21 Sep 2010 00:49:26 +0000 (00:49 +0000)]
Don't call opkg_conf_deinit() if opkg_conf_init() fails.
Pointed out by Sergey 'Jin' Bostandzhyan <jin@mediatomb.cc>.
git-svn-id: http://opkg.googlecode.com/svn/trunk@570
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower@gmail.com [Mon, 20 Sep 2010 23:56:11 +0000 (23:56 +0000)]
Disable sha256 code by default. This makes opkg GPLv2 by default.
git-svn-id: http://opkg.googlecode.com/svn/trunk@569
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Wed, 15 Sep 2010 00:39:22 +0000 (00:39 +0000)]
Add simple regression testing code and tests for some issues.
Written with python3 only because I was reading a tutorial and wanted
to do something useful with it.
git-svn-id: http://opkg.googlecode.com/svn/trunk@566
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Wed, 15 Sep 2010 00:05:37 +0000 (00:05 +0000)]
Use uppercase M for printing maintainer field, to be consistent.
From Sergey 'Jin' Bostandzhyan <jin@mediatomb.cc>.
git-svn-id: http://opkg.googlecode.com/svn/trunk@565
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Wed, 15 Sep 2010 00:05:24 +0000 (00:05 +0000)]
mask && PFM_FOO should instead be mask & PFM_FOO.
From Sergey 'Jin' Bostandzhyan <jin@mediatomb.cc>.
git-svn-id: http://opkg.googlecode.com/svn/trunk@564
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
google@wwsnet.net [Mon, 13 Sep 2010 01:14:38 +0000 (01:14 +0000)]
update manpages to reflect r559 and r562
git-svn-id: http://opkg.googlecode.com/svn/trunk@563
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
google@wwsnet.net [Mon, 13 Sep 2010 01:01:58 +0000 (01:01 +0000)]
- utilize opkg_conf_init() to initialize the pair lists before parsing the arguments
- implement --add-dest and --add-arch options to pass destinations and valid architectures via the commandline
git-svn-id: http://opkg.googlecode.com/svn/trunk@562
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
google@wwsnet.net [Mon, 13 Sep 2010 00:59:11 +0000 (00:59 +0000)]
- split the loading part of opkg_conf_init() out into opkg_conf_load()
- move the temporary destination list into the global config struct
git-svn-id: http://opkg.googlecode.com/svn/trunk@561
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
google@wwsnet.net [Mon, 13 Sep 2010 00:47:26 +0000 (00:47 +0000)]
fix compile issues on OS X and FreeBSD
git-svn-id: http://opkg.googlecode.com/svn/trunk@560
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
google@wwsnet.net [Mon, 13 Sep 2010 00:42:01 +0000 (00:42 +0000)]
implement --force-postinstall option, this allows forcing the execution of postinstall scripts in offline root mode
git-svn-id: http://opkg.googlecode.com/svn/trunk@559
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Tue, 24 Aug 2010 04:09:34 +0000 (04:09 +0000)]
Accept a leading "./" for the control file name, but don't mandate it.
git-svn-id: http://opkg.googlecode.com/svn/trunk@558
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Tue, 24 Aug 2010 04:09:21 +0000 (04:09 +0000)]
Remove stray semicolon.
git-svn-id: http://opkg.googlecode.com/svn/trunk@557
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Thu, 19 Aug 2010 01:54:04 +0000 (01:54 +0000)]
Add Recommended packages to the depended_upon_by field.
This ensures recommended packages are not orphaned prematurely. Minor code
clean up while here.
git-svn-id: http://opkg.googlecode.com/svn/trunk@556
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Thu, 19 Aug 2010 01:53:52 +0000 (01:53 +0000)]
Ensure Pre-Depended and Recommended packages can be autoremoved.
git-svn-id: http://opkg.googlecode.com/svn/trunk@555
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Wed, 18 Aug 2010 05:40:28 +0000 (05:40 +0000)]
Add generated man pages to .gitignore.
git-svn-id: http://opkg.googlecode.com/svn/trunk@554
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Wed, 18 Aug 2010 05:40:17 +0000 (05:40 +0000)]
Allow blacklisting a package in the status file.
To blacklist a particular package, add the following status entry for the
package to be blacklisted:
Status: deinstall hold not-installed
Should fix the opkg end of Issue #46. OpenEmbedded will need to correctly
prime the status file.
git-svn-id: http://opkg.googlecode.com/svn/trunk@553
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Wed, 18 Aug 2010 03:39:02 +0000 (03:39 +0000)]
Remove trailing whitespace. Sorry if this breaks your patches.
git-svn-id: http://opkg.googlecode.com/svn/trunk@552
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358
graham.gower [Wed, 18 Aug 2010 02:27:08 +0000 (02:27 +0000)]
Remove outdated comment regarding an old bug.
I managed to track down the "bug 1277" this comment referred to.
http://web.archive.org/web/
20050503234854/http://handhelds.org/~bugzilla/show_bug.cgi?id=1277
This referred to the status file containing incorrect information after a
failed upgrade.
git-svn-id: http://opkg.googlecode.com/svn/trunk@551
e8e0d7a0-c8d9-11dd-a880-
a1081c7ac358