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

2013-01-14 Thread Sandro Tosi
Hi again,

On Sun, Jan 13, 2013 at 5:16 PM, Sandro Tosi mo...@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


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



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

2013-01-14 Thread Paul Tagliamonte
On Mon, Jan 14, 2013 at 04:33:44PM +0100, Sandro Tosi wrote:
 Hi again,
 
 On Sun, Jan 13, 2013 at 5:16 PM, Sandro Tosi mo...@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 highly doubt that I'd discard your input wholesale :)

 
 - 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

  ACK. Not a bad idea.

 - i'd remove the vetriol from nonsense captcha :)

  Yeah yeah, sorry :) It was an attempt to be funny. I'll fix that.

 - 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.

  So, yeah. We talked about this. It'd be fairly trivial to add that to
  the dput-ng path, but there was some sort of discussion. Can't
  remember what. It's back on the stack.

 - 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)

   json objects are unordered, so that'd be fine :)

 - s/name/description/ in tweet.json + it should contain the post:
 true = committed as 9224aea

  d'oh! That file predates that (and I think the pre / post syntax
  sucks anyway, we should have order: pre or order: post or
  something)

  Thanks for pushing the fix!

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

\o/

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

Thanks for testing, Morph,
  Paul

-- 
 .''`.  Paul Tagliamonte paul...@debian.org
: :'  : Proud Debian Developer
`. `'`  4096R / 8F04 9AD8 2C92 066C 7352  D28A 7B58 5B30 807C 2A87
 `- http://people.debian.org/~paultag


signature.asc
Description: Digital signature


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

2013-01-13 Thread Paul Tagliamonte
tags 697768 + pending
thanks

If you would be so kind as to check over[1], that'd be absolutely
amazing.

Thanks so much,
  Paul

[1]: 
http://anonscm.debian.org/gitweb/?p=collab-maint/dputng.git;a=blob;f=examples/hooks/twitter/README.Installing;hb=HEAD

-- 
 .''`.  Paul Tagliamonte paul...@debian.org
: :'  : Proud Debian Developer
`. `'`  4096R / 8F04 9AD8 2C92 066C 7352  D28A 7B58 5B30 807C 2A87
 `- http://people.debian.org/~paultag


signature.asc
Description: Digital signature


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

2013-01-13 Thread Gergely Nagy
Paul Tagliamonte paul...@debian.org writes:

 If you would be so kind as to check over[1], that'd be absolutely
 amazing.

Read it through, was straightforward, will test the results once I have
something to upload. Thanks for the HOWTO!

-- 
|8]


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



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

2013-01-13 Thread Paul Tagliamonte
On Sun, Jan 13, 2013 at 11:16 AM, Sandro Tosi mo...@debian.org wrote:

 Hi Paul,

 On Sun, Jan 13, 2013 at 4:52 PM, Paul Tagliamonte paul...@debian.org
 wrote:
  If you would be so kind as to check over[1], that'd be absolutely
  amazing.

 I read the doc and that matches exactly what I did to experiment with
 posting to twitter before submitting this bug, so I can say it's good
 to go (I'll check it for real when I'll have a package to upload. Oh
 now that I remember... ;) ).

 Cheers  thanks for your work,


It's my pleasure!

I also just remembered that I have to document installing twitter into a
place where dput-ng can find it; that's coming in that file in the next 20
minutes, and a full hook install guide (for your other bug) shortly after.

One sec, hang tight,
(Thanks for looking!)
  Paul


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




-- 
:wq


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

2013-01-13 Thread Sandro Tosi
Hi Paul,

On Sun, Jan 13, 2013 at 4:52 PM, Paul Tagliamonte paul...@debian.org wrote:
 If you would be so kind as to check over[1], that'd be absolutely
 amazing.

I read the doc and that matches exactly what I did to experiment with
posting to twitter before submitting this bug, so I can say it's good
to go (I'll check it for real when I'll have a package to upload. Oh
now that I remember... ;) ).

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


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



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

2013-01-13 Thread Paul Tagliamonte
On Sun, Jan 13, 2013 at 11:18 AM, Paul Tagliamonte paul...@debian.orgwrote:



 On Sun, Jan 13, 2013 at 11:16 AM, Sandro Tosi mo...@debian.org wrote:

 Hi Paul,

 On Sun, Jan 13, 2013 at 4:52 PM, Paul Tagliamonte paul...@debian.org
 wrote:
  If you would be so kind as to check over[1], that'd be absolutely
  amazing.

 I read the doc and that matches exactly what I did to experiment with
 posting to twitter before submitting this bug, so I can say it's good
 to go (I'll check it for real when I'll have a package to upload. Oh
 now that I remember... ;) ).

 Cheers  thanks for your work,


 It's my pleasure!

 I also just remembered that I have to document installing twitter into a
 place where dput-ng can find it; that's coming in that file in the next 20
 minutes, and a full hook install guide (for your other bug) shortly after.

 One sec, hang tight,
 (Thanks for looking!)
   Paul


Just pushed it up, it's a bit hackish, full guide coming shortly.

Thanks again!
  Paul




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




 --
 :wq




-- 
:wq


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

2013-01-13 Thread Sandro Tosi
On Sun, Jan 13, 2013 at 5:20 PM, Paul Tagliamonte paul...@debian.org wrote:
 Just pushed it up, it's a bit hackish, full guide coming shortly.

It looks good (enough) for me - great! :)

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


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org