Bug 45568 - The server-file-transfer example is not very exemplary
Summary: The server-file-transfer example is not very exemplary
Status: NEW
Alias: None
Product: Ytstenut
Classification: Unclassified
Component: tp-ytstenut (show other bugs)
Version: unspecified
Hardware: Other All
: medium enhancement
Assignee: Robert Staudinger
QA Contact:
URL: https://github.com/r0b5t4/telepathy-y...
Whiteboard: review-
Keywords: patch
Depends on:
Blocks:
 
Reported: 2012-02-02 14:47 UTC by Robert Staudinger
Modified: 2012-02-03 03:29 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Robert Staudinger 2012-02-02 14:47:37 UTC
The  server-file-transfer example should also use a TpYtsClient when waiting to receive a file, so it can be discovered by other ytstenut apps and used for testing.
Comment 1 Robert Staudinger 2012-02-02 14:48:37 UTC
Patch, good to go?
Comment 2 Olli Salli 2012-02-03 03:29:05 UTC
The patch does multiple unrelated things in the same commit:
* make cosmetic changes to variable naming
* start using GOptions
* using it, make some formerly hardcoded variables specified on the command line
* invent new format for addressing the recipient service when sending
* finally, add the TpYtsClient when receiving (what this bug was specifically about)

I'm not saying these changes are bad; on the contrary, they are sensible and superficially seem well executed. However, it's hard to review them all in one (especially because some things are just cosmetics and non-functional refactoring), and also the patch description becomes misleading.

Thus, please split to smaller commits resembling something like the above outline.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.