[dput-ng-maint] Bug#697768: dput-ng twitter plugin install guide pending

Sandro Tosi morph at debian.org
Mon Jan 14 15:33:44 UTC 2013


Hi again,

On Sun, Jan 13, 2013 at 5:16 PM, Sandro Tosi <morph at debian.org> wrote:
> I'll check it "for real" when I'll have a package to upload.

I'm following the guide, and here are some notes (they are mainly
coming from my personal taste, so you can freely disagree and discard
them):

- i'd like more "dput-ng (<unique-id>)" so I created the app with that
name, it's pretty hidden so we can create the name as we prefer
- i'd remove the vetriol from "nonsense captcha" :)
- given the OAuth token generated is specifically forged for the newly
created app, I don't think it's apporpriate to clutter ~ with another
hidden file with such a generic name like (.twitter.json), we can
either consider created it under ~/.config/dput-ng/ or in the
~/.dput.d/ dir, like config or in the scripts subdir so that we can
read from it within the code - what do you think about it?
- would you consider moving the configuration for dput-ng under
~/.config ? it's a new app, and the ~/.config is being populated by a
lot of new apps, so you might want to consider that too.
- in twitter.json, I'd insert oauth_token before oauth_secret (at
least that's how it's proposed on the twitter page, so we can follow
that order here too)
- s/name/description/ in tweet.json + it should contain the "post":
true => committed as 9224aea

It seems that's all, it worked!
https://twitter.com/sandrotosi/status/290842762088173568 :)

Cheers,
--
Sandro Tosi (aka morph, morpheus, matrixhasu)
My website: http://matrixhasu.altervista.org/
Me at Debian: http://wiki.debian.org/SandroTosi



More information about the dput-ng-maint mailing list