config.c: fix up some compiler warnings

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Revert "turn on some better compiler warnings."

This reverts commit 78188317f96392e12e51e02f9a94150f0ac1786f.

Turns out that autogen.sh generates this properly, I had forgotten to
run that.

Thanks to L. Alberto for pointing this out.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

rewrite the config parser

Instead of a hand-tuned string compare functions that were needed
for every single config option, use a "real" parser to properly handle
key/value pairs.  This handles quoting of lines and other fun things.

Logic is based on code from udev, written by Kay Sievers, licensed under
the GPLv2+.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

fix compiler warning with bti_output_line

Stupid, forgot to mark it as a void function.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

move debug to bti.h

This is so that other .c files can call dbg();

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

turn on some better compiler warnings.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

release 029

move output logic to one function

This is in preparation for colored output.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

add config.c

This splits the config parsing logic out to a single file so we all
don't have to go blind by accidentally looking at the horror that it
imparts on our soul if we notice it.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

create bti.h

Moved the main data structure into there, this will let us
(hopefully) start refactoring the code to be a bit more readable
and maintainable over the long-run.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

move verbose flag into the session structure

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

reformat README to have a bit shorter lines

Merge branch 'oauth-readme' of https://github.com/agimenez/bti into agimenez-oauth-readme

* 'oauth-readme' of https://github.com/agimenez/bti:
  Document how to configure OAuth.

Merge testing branch into master

Merge https://github.com/leif81/bti into main tree

This fixes an issue with twitter queries for non-oauth portions
of the api.

updated man pages generated from the docbook source

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Active --dry-run for OAuth.

Guard the oauth_http_post/_get with an '!session->dry_run'
conditional, and add a dbg() statement before.

http://bugs.debian.org/608608

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Document the new --retweet feature in bti.xml and add it to bash completion.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Add documentation for new shrink_host feature in bti-shrink-urls.

Document the configuration of bti-shrink-urls in bti-shrink-urls.xml,
bti.xml, and add a commented out example to bti.example.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Document how to configure OAuth.

Signed-off-by: L. Alberto Giménez <agimenez@sysvalve.es>

user and public actions may not require auth

Support proper 'reply_to' and 'retweet'

Signed-off-by: Dwi Sasongko S <ruckuus@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Support shrinking with bit.ly and j.mp.

To use bit.ly or j.mp you need to specify the host name in ~/.bti with
shrink_host, your bit.ly login with shrink_bitly_login and your bit.ly key with
shrink_bitly_key.  You can register at:

  http://bit.ly/a/sign_up

If you already have an account, you can find your API key at:

  http://bit.ly/a/your_api_key

Fixing Identi.ca's OAuth request_token URI

Revision "a" of OAuth 1.0 mandates the presence of "oauth_callback"
parameter.

Usually libraries appends an "oob" (Out of Band) oauth_callback if you
don't provide a valid callback url, and don't specify that you don't
want "OAuth 1.0a".

But apparently our library of choice for OAuth (liboauth) does not do
this, though the code is in library but commented out.

src/oauth.c:675-680 (as in HEAD, and 0.9.0 tarball):

675 #if 0 // oauth_version 1.0 Rev A
676   if (!oauth_param_exists(argv,argc,"oauth_callback")) {
677     snprintf(oarg, 1024, "oauth_callback=oob");
678     oauth_add_param_to_array(argcp, argvp, oarg);
679   }
680 #endif

So for now, we have to add this to our request_token URI.

Update my copyright date.

release 028

Reduce the number of calls to fprintf.

Thanks to Flameeyes for the original patch, just changed it a bit
to work with the latest tree and to handle the version as a string.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Update my copyright date.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Collapse more fprintf() calls.

Split --background from --bash, and make the latter imply the former.

This allows to use send-in-background without mangling the message's text.

Typo fix.

Use arrays rather than pointers for static strings.

While this change does not do anything on most common systems, as
executables are not PIC, it is very useful when dealing with Position
Independent Executables (PIE) as used for instance by Gentoo Hardened.

Remove autoscan-generated bad tests.

build system: use AC_SEARCH_LIBS rather than assuming -ldl is needed.

On a number of operating systems, such as the *BSD, the dlopen() function
is present in the standard C library, not in a separate libdl library.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

build system: use consistently pkg-config to check for dependencies

Rather than rely on the old m4 macros for libxml2 and libcurl, which were
also misused, always use PKG_CHECK_MODULES (also assuming a new enough
version that calls AC_SUBST by itself), and fix the Makefile.am to use
_CFLAGS properly as well.

Signed-off-by: Diego Elio 'Flameeyes' Pettenò <flameeyes@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

long line cleanups

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

more formatting cleanups

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

whitespace cleanups for bti.c

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

new version of checkpatch.pl from 2.6.35-rc1 kernel

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Fixup typo in Makefile.am found by @applehq

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

release 027

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

fix up the autoconf checking for liboauth and other libraries

Start using pkg-config as its supposed to be used (hopefully)

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

- Cosmetics
- Escaping tweet before POSTing it

Merge branch 'master' into oath-test

Conflicts:
bti.c

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Leftovers from last commit, accepting account and, password from options

HTTP basic auth for StatusNet instances

Merging back all HTTP basic auth codebase, mostly needed for custom
StatusNet instances with no OAuth service provider.

Fix a warning: unused variable message

"file" became unused in commit 6fe2642650486b6ca6c2a6eadbd50246fbdd2edd

Add a replyto configuration option

Support for replying to notices

Allow the user to specify a single ID so that interfaces can show the
nice "in reply to" links (or "in context" for status.net installations).

Verbose mode shows status IDs

error out if liboauth is not present

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

OAuth for both twitter, and identi.ca

It's a bold move from the old HTTP auth to only OAuth. Since twitter is
going to drop http auth in next 10 days, and identi.ca has an OAuth
service provider, it seems dropping HTTP auth support from bti isn't
that bad.

You need to add your consumer key, and secret in ~/.bti:
consumer_key=yourkey
consumer_secret=yoursecret

bti will help you to get your "access token key", and "access token
secret". Put them in your ~/.bti and you're ready to go.

Merge branch 'master' into oath

Fix Implicit linking against -ldl

add oath.keys from twitter.com

release 026

Add ability to override default config file option

Thanks to Reuben Thomas <rrt@sc3d.org> for this idea.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

add a configfile option to the core data structure

In anticipation of adding the option to select where your config
file is at.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

init the readline function after parsing the command line options

This lets us properly debug the readline function choice if
we need to.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

update bti.1 from the xml changes

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Typo fixes for bti(1)

another minor codingstyle fix

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

updated version of checkpatch.pl

taken from Linux 2.6.34-rc5

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

minor coding style cleanup.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

bti: fix a crash for echo -n|bti

My previous patch was not thought out completely and so the new bti
crashes if fed an input without a newline, like this:
 echo -n "status test" | bti

The obvious fix seems the most appropriate approach in this case.

Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

bti: suppress prompt if fed from a pipe

Do not use readline for non-interactive sessions (e.g. those taking
input from a pipe or file). This way stray prompts are not shown.
Explicit prompts are retained, in case some script expects them.

Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

release 025

Remove link-time dependancy on readline

This patch divorces bti from depending on readline.  It now
will try to work with any readline-like library on your system,
and if none is found, fall back to an internal function.

This will solve the problems that distros are having with the
readline-6 license change to GPLv3 and bti staying at GPLv2.

Logic for this is taken from e2fsprogs, which implemented this
a long time ago for the same type of reason.

Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

Name correct hosts in prompts.

The prompt "Enter twitter {account,password}" is a bit misleading
if host is set to identi.ca or a custim host.
Add hostname to the session struct and use in for prompting and logging.

Reported as http://bugs.debian.org/563099

Merge remote branch 'gregkh/master'

Removing explicit mentions of twitter

If there's no host specified, the session host is already pointing to twitter, but hosturl is empty

release 024

use "quiet" mode of building if present.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

fix up some compiler warnings in read_password

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Disable echo when reading the password

fix crash when --debug is set

The group should not be printed out at startup if debug is set,
as it is not always set.  This came in in the "add group support"
patch, it was probably used for debugging that code.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

don't free an array on the stack

more coding style cleanups

clean up some trailing whitespace

BTI should not display password while user types it

Add password handling logic to not display the password on the
command line.  Code is based on logic from:
http://dev.kryo.se/iodine/browser/src/common.c

fix compiler warning about fprintf

Add laconica group support

As you offered on Identi.ca in reply to my question on Laconica group
support (http://identi.ca/notice/6205615), created a patch for it.
Been like 6 years when i learned and wrote my last basic C code, but
tested it out and seems to be working fine.

Signed-off-by: Marvin Vek <laen@onedot.nl>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

release 023

Updating AUTHORs and copyright section

Updating documents, both for the new verbose mode, and custom laconi.ca installations.

verbose option for bash completion script

Verbose mode for bti

https for twitter

release 022

Merge branch 'master' of git://github.com/amir/bti

bti: use ssl for identi.ca

Merge branch 'master' of gregkh@/linux/kernel/git/gregkh/bti

fix build warning from time patch.

bti: show timestamp in statuses

Make parse_statuses() print timestamps

Signed-off-by: J.R. Mauro <jrm8005@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Updating config example file

Support for custom bti installations

Now one can pass the URL of a custom laconica installation as value of
host (the both previous "twitter" and "identica" are still valid
though). The passed URL should be the base path of where API endpoints
are located. e.g., identica' baseurl is:
  http://identi.ca/api/statuses
And twit.tv's army (which is a laconica instance) is:
  http://army.twit.tv/api/statuses

add the script to the tarball.

release 021

bump version to 021