Re: Bug#802595: ITP: node-defined -- return the first argument that is `!== undefined`
On 10/23/2015 03:14 PM, Steve McIntyre wrote:
> Josh Triplett wrote:
>> Steve McIntyre wrote:
>>>
>>> YA tiny Javascript "library" containing 3 lines of utterly trivial
>>> code. :-(
>>>
>>> I appreciate you're just following through a dependency chain from
>>> upstream for tape, but please push back on upstream and ask them why
>>> they're doing this kind of ridiculous split-up. Code re-use in general
>>> is a good plan, but not at the level of every trivial helper function
>>> being split out into its own library!
>>
>> "why" is because node (and other modern languages) make it easy to
>> create a package for any particular bit of reusable code. That Debian
>> fails to support that is Debian's problem, not upstream's.
>
> In the general case I might agree, but have you actually looked at
> some of these cases? It's ridiculous in any language to have a
> separate library for a single function as trivial as:
>
> for (var i = 0; i < arguments.length; i++) {
> if (arguments[i] !== undefined) return arguments[i];
> }
>
> Split it out into a separate helper function in the surrounding code?
> Sure. Add it to your own library with lots of other little helpers?
> Yes, by all means if you're using it a lot. But a separate library
> with its own docs and test suite and everything? No, that's a joke.
+1
But good luck to teach good practices upstream. See Ross's reply: 120
packages are depending on this.
Though it is also my view that packaging tiny stuff shouldn't be a
problem. If it is, then we should fix whatever it is that is problematic
in Debian infra.
Cheers,
Thomas Goirand (zigo)
Reply to: