view_app_file(), view_readmes(): This puts the dep info back into the
README display by cat'ing the README and grep'ing the .info file into a
temporary README. This doesn't touch the command line show_readme() as
that actually shows both the README and .info files already.
Thanks to Joni Martikainen for the suggestion.
(While editing the NEWS file, I also moved the -r item to FEATURES
since, while legal, -r combos weren't useful and now they are, so I
guess it's more a featureling than a modification.)
This primarily revises start_dialog_queue() to provide a download-only
dialog UI.
* global: make many more minor textual changes which replace 'build' and
'build queue' references with more general references including
'queue'
* info_item(): rename the 'Build' option to 'Process' (moving 'build'
into the new description of 'Download/build/install') as it always
could build or install and can now download, too.
* add_item_to_queue(),process_queue(): tweaked size of dialog widget
* start_dialog_queue(), process_queue(), main: rewrite
start_dialog_queue() as a menu which provides the download-only option
in addition to build, install, and cancel. Combine the two error
handling instances in it (which would have been three) and the one in
main into one chunk in process_queue().
* usr/doc/README{,-queuefiles}, usr/man/man*: more 'build'-related
textual tweaks
* usr/doc/THANKS: remembered to add the r905 RFE credit
Thanks to Mauro for discussion and review.
Make changes to reflect the logic that "'install' includes 'build'
includes 'download'" vs. "'build' and 'buildinstall' are separate
alternatives, both of which implicitly assume 'download'":
* Expand {QUEUE,}TYPE's possible values to include
'download|build|install' and create set_type() to set it in place of
'build|buildinstall' and using the DOWNLOADONLY variable and the
{B,I}FLAGs.
* main: delete {B,I}FLAG from the 'unset' section and from the option
parsing
* main, set_type(): break out the option testing error message and
TYPE assignments into set_type() which, rather than assigning to
TYPE in one place and testing {B,I}FLAG in another, assigns to TYPE
unless TYPE is already set and has a different value, in which case
we error out. (People should theoretically be able to call -d, -b,
-i and have each trump the previous ('-b foo -i foo' should
download, build, and install foo), but calling '-b foo -i bar' is
problematic at best.)
* process_queue(),start_dialog_queue(): replace 'buildinstall's with
'install's
* Clean up sbopkg's download/build/install logging and output:
* check_source(): log that we've found the source and return if we're
only doing a download.
* process_package(): s/Building/Processing/; use pre-existent
QUEUETYPE variable with new possible value of 'download'; say we've
downloaded and return 1 rather than just breaking.
* log_queuetype(): create by breaking out some redundancy in
process_queue() relating to the PRE-CHECK and SUMMARY logs and make
it handle 'Download only' in addition to build/install.
* process_queue(): remove references to 'PACKAGE BUILDING/INSTALLING'
because it can be downloading also and 'PACKAGE
DOWNLOADING/BUILDING/INSTALLING' seems excessive; use
log_queuetype()
* main: switch the -d and -D flags. It's not good to shuffle flags
generally, but -d is probably rarely used and download would better be
lowercase to be consistent with -b/uild and -i/nstall.
* main: while messing with options, sorted the getopts list, the option
parsing, and the help output as though it had been sorted with 'sort
-f' (previously, capitalized options were inconsistently sorted).
* src/usr/doc/NEWS,src/usr/man/man8/sbopkg.8: document user-visible
aspects of the above (-d/-D), tweaking the DESCRIPTION and the -b, -i,
-k and -R flags in the manual.
* build_package(): rename the bulk to process_package() and split out a
chunk as a new build_package(), causing a break out of the function if
DOWNLOADONLY is set and calling build_package() if not.
* build_package(): create from a piece of the earlier build_package().
* process_queue(): call process_package() rather than build_package().
* main: add DOWNLOADONLY and the -D flag.
* various: modify comments to reflect current code
Thanks to Mauro Giachero for this patch. :)
Don't assume the reader knows what "queuefiles", "rename files", or
"repo files" are (even though repos and queues are briefly mentioned in
"ABOUT"). Instead, describe what they're for.
check_for_updates(), view_readmes(), search_package(), string_search(),
remove_obsolete_sources(): add mindepth/maxdepth flags (using a DEPTH
variable in string_search() (because it varies) and a FIND_FLAGS
variable in check_for_updates() (because it's reused)) to the find
commands that operate in the REPO. This will avoid matching things or
trying to use things like openvswitch/xen/foo when xen/foo is intended.
Thanks to David Somero for reporting this.
and additional VERSION parms. Now examples in sbopkg(8) and README-repos
which depend on Slackware/SBo versions won't have to choose between
being updated manually or looking out-of-date but will be updated
automatically. The sbopkg VERSIONs are just to be more consistent with
other usage - while svn users won't get to have manual pages saying
'SVN', the manuals' talking about an sbopkg/SVN dir were wrong and the
other uses were at least inconsistent and it's all vcs-specific anyway.
This revision is a tip of the iceberg to a larger backend release
process change. Which. will. happen. any. day... now....
* sbopkg.conf.5: removed the reference to the -O flag in the WGETFLAGS
section, which hasn't been true since 0.20.0. I caught this in
sbopkg.conf almost a year ago in r817 but kept missing it here. Also
clarified where what-would-be-the-TMP-files go when CLEANUP is set
and TMP is ignored.
* sbopkg.8: two trivial tweaks.
i.e., rename usr/doc/queuefiles/* to usr/doc/contrib/queuefiles/*
usr/contrib will provide a place to add other contrib items, such as
bash completions.
The assignment to SRCNAME was trying to cut out the longest match of
'%/download' rather than cutting off '/download'. The notable practical
effect was not recognizing that we had a source in the cache, since
foo.tar.gz/download != foo.tar.gz, so downloading it again. (Again, I
wonder how this could have ever gotten through testing and, again, I
feel like this sort of botched parameter expansion has happened before,
but can't find it in the vcs.)
Sure seems like I tested this and it worked but I guess I focused on
view_queue() and list_packages() at the wrong time. I knew
remove_files() (via view_cache_dir()) had a quoted list since source
names can include any kind of lunacy, unlike package names, but forgot
to allow for it. So allow zero or more dquotes around LINE.
A cluster of changes related to creating symlinks in the
/etc/{renames,repos}.d directories that point to the corresponding
README in sbopkg's docdir and then generally updating most
build-related files and getting it all documented.
* 50-default: renamed to 50-default.renames
* NEWS: note symlink docs and rename issues
* README-renames: update to reflect new default filename and the fact
that the files must end in '.renames'.
* sbopkg.conf.5, sbopkg.8: update to reflect the new renames stuff
(and the earlier repo file/dir-related changes I forgot to document
there).
* sbopkg: get_new_name(), get_old_name(): have grep only look through
*.renames files and not, say, symlinks to READMEs.
* doinst.sh: bring into alignment with the current SBo template.
* tools/sbopkg.SlackBuild: make links from README-repos and
README-renames to READMEs in the corresponding /etc/sbopkg/*.d
directories.
* tools/slack-desc: bring into alignment with the current SBo
template.
* tools/sspm: update address, add printe() and use it.
If we keep the Id keyword in sbopkg's config file, users having their
'sbopkg.conf' Id'ed as 'sbopkg.conf.new' can cause confusion, as seen on
LQ. So call it 'sbopkg.conf' in revision control and append '.new' when
creating the package.
