pudwerkin / udpt

Automatically exported from code.google.com/p/udpt
0 stars 0 forks source link

some thoughts about udpt project #5

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I plan to run a small retracker and I found that udpt perfectly suits my needs. 
Thanks for the brilliant idea (small UDP tracker wihout «fill-sized RDBMS» 
backend) and good work!

Some issues after 1st build and run:
1. Seems that makefile does not have install phase and udpt binary but be 
installed manually.
2. Some mess with config file – this file (together with db file) created in 
current directory, which is very strange :) More, path_to_config as first 
argument is ignored and some parsing errors are detected (If you're interested 
I'll investigate this further). I expected more or less «standard» 
3. Logging. Stdout only? Some *funny* messages like «E:(null)» or «[not an 
error]» – is not so obvious :)

By the way, I've created simple ebuild for Gentoo – if you think to put it 
together with debian packages, I'll gladly share it.

Original issue reported on code.google.com by beelzebu...@gmail.com on 10 Aug 2013 at 4:30

GoogleCodeExporter commented 9 years ago
one addition – bind to 0.0.0.0 is fine but it would be very nice to have a 
configurable value

Original comment by beelzebu...@gmail.com on 10 Aug 2013 at 4:33

GoogleCodeExporter commented 9 years ago

Original comment by naim...@gmail.com on 10 Aug 2013 at 5:50

GoogleCodeExporter commented 9 years ago
Please provide the config file that causes parsing errors.

Original comment by naim...@gmail.com on 10 Aug 2013 at 6:12

GoogleCodeExporter commented 9 years ago
http://bpaste.net/show/121746/

сauses 

terminate called after throwing an instance of 'std::logic_error'
  what():  basic_string::_S_construct null not valid
Aborted

Original comment by beelzebu...@gmail.com on 10 Aug 2013 at 7:28

GoogleCodeExporter commented 9 years ago
Another one config-related bug:
docs says: «If no configuration file is present, a default configuration file 
will be created by the program (if a parameter was specified, it will be 
created there).», but it is not:

# udpt /tmp/udpt.conf
UDP Tracker (UDPT) 1.0.0-beta (Linux)
Copyright 2012,2013 Naim Abda <naim94a@gmail.com>
    Released under the GPLv3 License.
Build Date: Aug 10 2013

Failed to read from 'udpt.conf'. Using default settings.
Starting maintenance thread (1/6)
Starting thread (2/6)
Starting thread (3/6)
Starting thread (4/6)
Starting thread (5/6)
Starting thread (6/6)
Press Any key to exit.
^C
# ls -l /tmp/udpt.conf
ls: cannot access /tmp/udpt.conf: No such file or directory

Original comment by beelzebu...@gmail.com on 10 Aug 2013 at 7:37

GoogleCodeExporter commented 9 years ago
I've just fixed that with the last push to the repo.
Thanks!

Original comment by naim...@gmail.com on 10 Aug 2013 at 7:43

GoogleCodeExporter commented 9 years ago
can not build after this push: http://bpaste.net/show/121848/

Original comment by beelzebu...@gmail.com on 11 Aug 2013 at 7:45

GoogleCodeExporter commented 9 years ago
That was a Makefile problem... fixed.

Original comment by naim...@gmail.com on 11 Aug 2013 at 9:24

GoogleCodeExporter commented 9 years ago
comment lines where supposed to start with a semicolon (';') not with hashes 
('#'), but i thought it was a good idea - so i added hashes as comments.
Please approve fixes (commit db74edbb51df).

Original comment by naim...@gmail.com on 11 Aug 2013 at 10:24

GoogleCodeExporter commented 9 years ago

Original comment by naim...@gmail.com on 14 Aug 2013 at 12:15

GoogleCodeExporter commented 9 years ago
Config file is now read from executable's directory (if non specified) instead 
of current directory.

Original comment by naim...@gmail.com on 15 Aug 2013 at 9:55