{"issue":{"title":"Changes for embedded system support","user":{"login":"chuckb","id":239663,"url":"https://api.github.com/users/chuckb","avatar_url":"https://secure.gravatar.com/avatar/8bcc9bee57dd688275afb54e0a06f2c4?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"8bcc9bee57dd688275afb54e0a06f2c4"},"created_at":"2011-12-29T06:36:16Z","milestone":null,"state":"closed","updated_at":"2011-12-29T14:44:31Z","labels":[],"id":2677549,"closed_at":"2011-12-29T14:44:31Z","html_url":"https://github.com/Flameeyes/scan2pdf/issues/4","pull_request":{"patch_url":"https://github.com/Flameeyes/scan2pdf/pull/4.patch","html_url":"https://github.com/Flameeyes/scan2pdf/pull/4","diff_url":"https://github.com/Flameeyes/scan2pdf/pull/4.diff"},"body":"- Throttles concurrent requests for memory constrained system.\r\n- Ditches qpixel step in unpaper for memory constrained system.\r\n- Plays nice with Debian squeeze packages (sane < 1.0.22).","number":4,"url":"https://api.github.com/repos/Flameeyes/scan2pdf/issues/4","comments":1,"assignee":null},"comment":{"user":{"login":"Flameeyes","id":74834,"url":"https://api.github.com/users/Flameeyes","avatar_url":"https://secure.gravatar.com/avatar/7298d3f690c8bdf6b3bd628b6bef1f91?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"7298d3f690c8bdf6b3bd628b6bef1f91"},"created_at":"2011-12-29T14:44:31Z","updated_at":"2011-12-29T14:44:31Z","id":3301149,"body":"I have rewritten your third commit (to support sane-backends 1.0.21) to use a proper version comparison (your code would have broken with 2.0.0 and 1.3.0), but I'm not really a fan of the other two.\r\n\r\nFirst, the use of `free` as a check sounds like a very bad idea to me, especially toward portability. I've done my best for this to run on any posix-compatible system where sane-backends could run, I don't think free is portable. Plus I dislike relying on output from procps utilities, considering Debian is taking it over for good.\r\n\r\nAs a final touch, it'll be waiting for my box to have 5GB of free memory... that's never going to happen on a normal day.\r\n\r\nWhat I plan on doing next is adding a `-Wu` option that allows passing options directly to unpaper, so that `-Wu,--no-qpixels` would work without otherwise changing `scan2pdf` — that combined with the configuration file support I just pushed should do the trick for you about the qpixels problem.\r\n\r\nAs for the multiple processes at once, I need to find a way to implement a `-j` option akin to `make`'s which allows setting how many jobs at once to proceed with.","url":"https://api.github.com/repos/Flameeyes/scan2pdf/issues/comments/3301149"},"action":"created"}