sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: x/build/cmd/gopherbot: -2 any CLs with large binary blobs unless declared in commit message
Labels: help wanted,Builders,NeedsInvestigation,FeatureRequest

See CL 9560 for context.

There will be cases in which it is legit to add a binary file. In such cases, we could do a force commit. Other suggestions welcomed.

**Edit by @dmitshur:** This also happened in [CL 149604](https://golang.org/cl/149604), see #28899.
Repository: rust
Issue: type parameter not constrained when using closure bounds
Labels: C-enhancement,A-diagnostics,T-compiler,D-confusing,D-newcomer-roadblock,F-unboxed_closures

This works fine:

``` rust
trait Foo {}
impl<F, A> Foo for F where F: Fn() -> A {}
```

But this:

``` rust
trait Foo {}
impl<F, A> Foo for F where F: Fn(A) {}
```

produces a compile error:

```
<anon>:2:9: 2:10 error: the type parameter `A` is not constrained by the impl trait, self type, or predicates [E0207]
<anon>:2 impl<F, A> Foo for F where F: Fn(A) {}
```

I'm not sure if this is intended behavior or not, but I definitely don't understand why the former is accepted and the latter is not.

Version:

```
[andrew@Liger quickcheck] rustc --version
rustc 1.1.0-nightly (c4b23aec4 2015-04-29) (built 2015-04-29)
```
Repository: neovim
Issue:  Use absolute line to display debug info ?
Labels: enhancement,vimscript

content of `testrc`

```
"Test relative line
func! Test()
    let abc = undefined_variable
endf

call Test()
```

run `nvim -u testrc`

```
Error detected while processing function Test:
line    1:
E121: Undefined variable: undefined
E15: Invalid expression: undefined
```

Would Neovim use absolute line to display debug info ?
eg:

```
Error detected while processing function Test:
line    3:
E121: Undefined variable: undefined
E15: Invalid expression: undefined
```
Repository: You-Dont-Know-JS
Issue: "this & object prototypes": ch2, fix misleading code/info
Labels: for second edition

In ch2, "Implicitly Lost", the last example suggests that `setTimeout(..)` causes the `this` to be lost (and thus bound to `window`) because of a theoretical call-site of `fn()`. However, this is not what `setTimeout(..)` actually does. It's verifiable that even if the example in question is run in `strict` mode, `"oops, global"` is still printed. That's because the web platform actually says that `setTimeout(..)` explicitly binds the `this` to the global object.

This is a case where the outcome is correct, but my explanation of how that happens is incorrect.

Technically, this is more what happens

``` js
function setTimeout(fn,delay) {
    // wait (somehow) for `delay` milliseconds
    fn.call(window); // <-- call-site!
}
```

However, this is not an _explicit binding_ in the "Implicitly Lost" section, which makes it out of place. Either update the code and text to be accurate, and then move it to the "Explicit" section somewhere, or (perhaps better?) just completely remove it. It doesn't really add much, TBH.

**Note:** node/iojs doesn't follow the web platform spec here, so the `this` they explicitly bind is _not_ the global. Ugh.
Repository: TypeScript
Issue: Compile / edit time pluggable analyzers like C#
Labels: Suggestion,Needs Proposal

It would be nice to have similar plugabble compile time / editing time analyzer plugins as [introduced in C# in Build 2015, see the video](http://channel9.msdn.com/Events/Build/2015/3-725).

Analyzer is a library that can do additional syntax checks, tag lines as errors / warnings, and are interpreted by language service.

I can imagine one could do similar things with it as Facebook's flow does e.g. own nullable checking just by writing own analyzer. This would be configurable from tsconfig, maybe one could whitelist the files each analyzer works on. This way the nullability check analyzer would only complain about the code I want.
Repository: rust
Issue: Overflow with use of `Self` in inherent impl on trait
Labels: T-compiler,C-bug,F-associated_type_defaults

Code: http://is.gd/zUrIKd

``` rust
pub trait NBTTypeHelper where Self: NBTType {
    type Tagtype = Self;
}

pub trait NBTType {
}

impl NBTTypeHelper for NBTType {
    type Tagtype = <Self as NBTTypeHelper>::Tagtype;
}

fn main() {}
```
Repository: rust
Issue: Unable to export not mangled symbol in an executable as libraries can do
Labels: A-linkage,A-visibility,C-bug

trying something like this:

``` rust
#[no_mangle]
pub extern "C" fn foo()
{
    //...stuff...
}
```

`foo()` seems to get exported if compiled inside a library, but if compiled in a binary crate it doesn't.
I found this out trying to use gtk builder's "gtk_builder_connect_signals", which requires for not mangled symbols to be exported.
Is this behaviour a bug? or is it a design choice? if so, why?
Repository: youtube-dl
Issue: [nba] Add support for authentication
Labels: site-support-request

Attempting to download video from http://watch.nba.com (eg `http://watch.nba.com/nba/games/20150501/ATLBKN/41400106`) resulted in an "unsupported UR" error.

The request was made with appropriate username and password credentials for the resource.

Is support for http://watch.nba.com different to http://nba.com, which is listed as a supported domain?
Repository: go
Issue: testing: show more benchmark data
Labels: NeedsInvestigation

There are a variety of analyses that would increase confidence in benchmark data. Package testing should _not_ do them, but it probably should make them possible, by having a mode to emit more data about the benchmark run than just the one number it does today.

@aclements
Repository: youtube-dl
Issue: [Generic] Add support for playlists if more than one video is found
Labels: request

Treat a url as a playlist if more than one video url is found. This should be a thing for every url that is handled with the generic video extractor.
Repository: gin
Issue: the Catch-All form should not include the "/" before param
Labels: enhancement

As the doc said, 

```
r.GET("/user/:name/*action"...
action := c.Params.ByName("action")
```

should return value string of **action**, but in fact it returns **/action** with the slash before param string, which is not correctly & inconvenient.
Repository: TypeScript
Issue: Rationalize our presentation and handling of merged/alias symbols in the IDE.
Labels: Suggestion,Help Wanted,API,Domain: Quick Info

Examples of where we could be better:
1. Quick info for an alias could present information about the alias, as well as information about what the alias points to.
2. Goto-type for a union type could allow you to go to any constituent union-type-element.
3. Goto-def on a merged symbol could allow you to go to any declaration.
4. Quick info on a merged symbol could show you all the constituent declaration pieces.

etc. etc.
Repository: youtube-dl
Issue: Annotations as subtitles
Labels: request

Would it be feasible, using the `lxml` library, to have Youtube-DL find a way to save annotations as subtitles? (Referring mainly to its usage in MPV via the youtube-dl hook, but also for downloading: for example, downloading a video's annotations and muxing them into a WebM as WebVTT or an MKV as ASS/SSA)

I understand that this feature might be a little complicated to implement, and might require some collaboration between the authors of Youtube-DL and MPV, but it would be a very nice feature to have.
Repository: rust
Issue: "type ... too big for the current architecture" error reported without span pointing to type
Labels: E-hard,A-diagnostics,T-compiler,C-bug

e053dfad23515f7020171ae18013b230531a6042 added checks for huge types. These checks are done in middle/trans, but the location of the type is not reported.

For anonymous array types there's no way to extract the span, as the ast-ID is lost.

An example:

``` rust
fn main() {
    let _x = [0; 9223372036854775808];
}
```

the error message:

```
error: the type `[i32; 9223372036854775808]` is too big for the current architecture
```
Repository: You-Dont-Know-JS
Issue: "this & object prototypes": Chapter 2 Clarification??
Labels: question,for second edition

I don't know if this makes sense or not.  As I was trying to digest this chapter, I got sent down a rabbit hole when the call stack was mentioned in finding the call-site.  I modified the example in the chapter to read like this.:
function baz() {
    var a = 1;
    // call-stack is: `baz`
    // so, our call-site is in the global scope

```
 console.log( "baz" );

 console.log(this.a);
 bar(); // <-- call-site for `bar`
```

}

function bar() {
   var a = 2;
   // call-stack is: `baz` -> `bar`
   // so, our call-site is in `baz`

   console.log( "bar" );
   console.log(this.a);
   foo(); // <-- call-site for `foo`
}

function foo() {
    var a = 3;
    // call-stack is: `baz` -> `bar` -> `foo`
    // so, our call-site is in `bar`

```
console.log( "foo" );
console.log(this.a);
```

}
var a = 0;
baz(); // <-- call-site for `baz`
console.log(this.a);

I confused this.a with the local var a in each function, thinking that I should see baz, 1, bar, 2, foo 3, 0 which would be the output if I my console.log statements were console.log(a) (printing out the local
variables in lexical scope for each function call).  

Perhaps, it was just my brain confusing this, but if you think otherwise such that others could likewise get confused here, then my suggestion would be to point this out with a similar example.
Repository: go
Issue: x/build: make GCE buildlet pool quota accounting smarter
Labels: Builders

The GCE implementation of BuildletPool's quota accounting is way too dumb & conservative.

If it counted better, it could push our quota limits a lot harder.

In fact, it could push things as hard as possible until it hit limits, learn the limits, and then count from there.

Or it could poll quotas?  (is there an API?)

Currently we have quota for 200 CPUs (and N RAM and disk, etc) but we limit ourselves to 60 builds at once, just because we haven't done the work to count how much of resource $X each build uses.

/cc @adg @crawshaw
Repository: go
Issue: x/build: builds waiting on resources shouldn't be in building state
Labels: Builders

/cc @crawshaw @adg 

The reverse BuildletPool impl is cheating a bit or interacting weirdly with the rest of the coordinator: instead of blocking on returning a buildlet, it instead "starts" a bunch of builds even if there's not an idle iPhone available.  And then each builds looks like it's in progress on build.golang.org (with happy blue gopher), but in reality they're all just stuck, for 4 hours (!?) here:

![screen shot 2015-05-06 at 9 04 59 am](https://cloud.githubusercontent.com/assets/2621/7497401/03c23cc2-f3cf-11e4-9471-dad60e920c2c.png)

Arguably, the GCE pool impl might do this too, but it happens much less often so it's been easier to miss and ignore.

We should have make quota a more first-class concept within the coordinator and have different states for "waiting" vs "building".  And when quota becomes available, we should prioritize which waiting build gets to go next.  Right now the one who gets lucky and grabs a mutex gets to go next.

Then, once we have two states, we should make build.golang.org have two colors of gophers: gray for waiting, and blue for actively building.
Repository: go
Issue: x/build: run both root and non-root Linux builders
Labels: Builders,FeatureRequest,new-builder

We should run at least non-root Linux builder to catch tests which accidentally require root.

(but we shouldn't run all builders as non-root, else we'll miss things assuming non-root)

Perhaps we make some existing weird builder also have this role, like linux-386-387, to not add resources or columns. We could rename it to 386-387noroot if we wanted?

/cc @minux @adg @crawshaw
Repository: go
Issue: gccgo: does not build on Darwin
Labels: NeedsInvestigation

Tried building via the procedure documented at https://golang.org/doc/install/gccgo using the gccgo branch. Hit a problem during configure. iant on the IRC informs me that Darwin is currently not supported. Not a blocker, but based on https://sourceware.org/gdb/wiki/GoDebugging the gccgo debugger support is fairly close to complete, and intellij and/or emacs with debug support might be handy in a pinch.

Longer-term LLDB might be the destination so maybe this is moot.

That said, checking out the source takes a while and there are a lot of devs using Darwin. It might be nice if the gccgo page said, up-front, which platforms were currently supported. Ideally in a manner which pulled from the actual install metadata. The configure might also detect this ASAP. Just saying. When Mr. Kernighan's book is released there will be a wave of people looking into the new-and-improved 'c'.
Repository: youtube-dl
Issue: Add if statement for output template
Labels: request

I currently use this: `-o "D:\Movies\youtube-dl\%(playlist_index)s.%(title)s-%(id)s.%(ext)s"` but whenever I download a video that's not in a playlist, it gets saved with a filename that starts with `NA.<title>`.

It would help if it was possible to check if some variable exists, like `-o "D:\Movies\youtube-dl\%if(playlist_index)s{%(playlist_index)s.}%(title)s-%(id)s.%(ext)s"`
Repository: rust
Issue: Order-dependent type inference failure with match
Labels: A-type-system,C-enhancement,T-compiler,T-types

Not sure if this has been reported yet, but it really seems like Rust should be able to do better here.

This also works if we replace the `match` with an `if let` and move the `res` assignment below it.

Note that this is a reduced testcase and therefore contrived, but the original issue can be seen [here](http://www.reddit.com/r/rust/comments/352u97/question_about_function_accepting_a_closure_as_an/).

---

``` rust
struct Foo;

impl Foo {
    fn foo(self) -> () { () }
}

fn main() {
    let mut res = None; // Compiles if we use `None::<Foo>`;
    match res {
        Some(winning) => { let _: () = winning.foo(); },
        None => res = Some(Foo), // Compiles if we move the None clause above the Some.
    }
}
```

``` console
<anon>:10:40: 10:53 error: the type of this value must be known in this context
<anon>:10         Some(winning) => { let _: () = winning.foo(); },
                                                 ^~~~~~~~~~~~~
```
Repository: go
Issue: x/tools/refactor: "Extract" features
Labels: Tools,Refactoring

This is a feature request, not a bug.

It would be nice to have a command that makes it easy to extract methods and functions from code.

Something like: a command that takes a range of code in a file that should be replaced with a function or method call and a name for that function or method (let's just call it "func"). 

The code in the given range would be replaced in the file with a call to the new func (with return values, too) and a definition of the func would be inserted into the current file next door to the current func (or, wherever the struct is defined in the current package if it's only in one place). The arguments and return value for the func would be inferred from the variables the code range uses.

It would be nice for the command to infer whether to build a function or method by looking to see if the range of code given is inside a method and if the code refers to the method's target.

More magic would be to allow the addition of functions and methods to structs in other packages with this tool.
Repository: kubernetes
Issue: FUSE volumes
Labels: priority/awaiting-more-evidence,sig/storage,kind/feature,needs-triage

I have an use-case where I would like to mount Google Cloud Storage (GCS) bucket (and a directory in that bucket) in my container and use it as a regular FS. Currently, it seems doable using s3fs-fuse (https://github.com/s3fs-fuse/s3fs-fuse) - thanks @brendanburns. It would be great if GCS was supported as a first class Volume in Kubernetes.
Repository: youtube-dl
Issue: [vimeo] Support private channels
Labels: request,account-needed

Hi there!

Some days ago I was able to download Videos from vimeo.com, but now I always get 404 error messages when I try it.

Output is this:

[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-f', u'h264-hd', u'-u', u'PRIVATE', u'-p', u'PRIVATE', u'-4', u'https://vimeo.com/channels/900773/126813964', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.05.04
[debug] Python version 2.7.6 - Linux-3.13.0-24-generic-x86_64-with-LinuxMint-17-qiana
[debug] exe versions: avconv 9.18-6, avprobe 9.18-6, rtmpdump 2.4
[debug] Proxy map: {}
[vimeo] Logging in
[vimeo] 126813964: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 312, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1708, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 410, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 523, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 448, in error
    return self._call_chain(_args)
  File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
    result = func(_args)
  File "/usr/lib/python2.7/urllib2.py", line 531, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Is this a youtube-dl problem or a problem on my side?
Repository: rust
Issue: lint “non_upper_case_globals” wrongly warns the consumer
Labels: A-lints,T-lang,C-bug

The lint “non_upper_case_globals” wrongly warns the consumer about a ill-named constant ([playpen](http://is.gd/GplrQb)). It only seems to happen if the constant is imported into the current namespace and if you’re using it in a match statement:

``` rust
mod consts {
    #![allow(non_upper_case_globals)]
    pub const fooBAR: usize = 23;
}

fn main() {
    use consts::*;
    println!("{}", match 23 {
        fooBAR => "yes",
        _ => "no"
    });
}
```
Repository: youtube-dl
Issue: Add support for http://triathlonlive.tv/
Labels: site-support-request
Repository: go
Issue: x/mobile/app: app crashes with DeadObjectException on Android 
Labels: mobile

Applications seem to be throwing an exception during pause, makes them crash on resume. The case is consistently reproducible with the basic example.
1. Launch the basic example on an Android device
2. Press back to pause it
3. Bring app to the foreground

Application crashes with the following log.

```
W/ActivityManager(  510): Exception thrown during pause
W/ActivityManager(  510): android.os.DeadObjectException
W/ActivityManager(  510):   at android.os.BinderProxy.transactNative(Native Method)
W/ActivityManager(  510):   at android.os.BinderProxy.transact(Binder.java:496)
W/ActivityManager(  510):   at android.app.ApplicationThreadProxy.schedulePauseActivity(ApplicationThreadNative.java:701)
W/ActivityManager(  510):   at com.android.server.am.ActivityStack.startPausingLocked(ActivityStack.java:852)
W/ActivityManager(  510):   at com.android.server.am.ActivityStack.finishActivityLocked(ActivityStack.java:2754)
W/ActivityManager(  510):   at com.android.server.am.ActivityStack.finishTopRunningActivityLocked(ActivityStack.java:2611)
W/ActivityManager(  510):   at com.android.server.am.ActivityStackSupervisor.finishTopRunningActivityLocked(ActivityStackSupervisor.java:2453)
W/ActivityManager(  510):   at com.android.server.am.ActivityManagerService.handleAppCrashLocked(ActivityManagerService.java:11511)
W/ActivityManager(  510):   at com.android.server.am.ActivityManagerService.makeAppCrashingLocked(ActivityManagerService.java:11408)
W/ActivityManager(  510):   at com.android.server.am.ActivityManagerService.crashApplication(ActivityManagerService.java:12087)
W/ActivityManager(  510):   at com.android.server.am.ActivityManagerService.handleApplicationCrashInner(ActivityManagerService.java:11603)
W/ActivityManager(  510):   at com.android.server.am.NativeCrashListener$NativeCrashReporter.run(NativeCrashListener.java:86)
```

cc/ @crawshaw @hyangah
Repository: kubernetes
Issue: Run services as non-privileged users
Labels: priority/backlog,area/security,kind/cleanup,sig/apps,help wanted,lifecycle/frozen

At the moment most services in kubernetes run as root. This includes (from glancing at ps aux on a minion):
- etcd
- kube2sky
- td-agent
- ~~skydns~~ (no longer exists in addons)
- elasticsearch
- ~~heapster~~ (removed in 31fb04fa98e34773e18bd32d5aa4dfc554a9b83e)

It's best practice to use non-root user accounts for services that don't need them. E.g. elasticsearch had remote code execution vulnerabilities in the past.
Even though docker's root has a capability bounding set applied so it doesn't translate to a real root on the host, the attack surface for privilege escalation attacks is still increased.
Repository: rust
Issue: Pretty Printing on `*-windows-gnu` broken because .debug_gdb_scripts is not emitted
Labels: A-debuginfo,P-medium,T-compiler,O-windows-gnu,C-bug,WG-debugging

```
rustc --version
rustc 1.0.0-beta.2 (e9080ec39 2015-04-16) (built 2015-04-16)

rustc -g main.rs
dumpbin /summary main.exe
Microsoft (R) COFF/PE Dumper Version 14.00.22816.0
Copyright (C) Microsoft Corporation.  All rights reserved.


Dump of file main.exe

File Type: EXECUTABLE IMAGE

  Summary

        1000 .CRT
        1000 .bss
        1000 .data
        2000 .debug_abbrev
        2000 .debug_aranges
        1000 .debug_frame
       1A000 .debug_info
       13000 .debug_line
        1000 .debug_pubnames
        1000 .debug_pubtypes
       11000 .debug_ranges
        1000 .debug_str
        1000 .edata
       24000 .eh_frame
        2000 .idata
       3F000 .rdata
        C000 .reloc
       CC000 .text
        1000 .tls
```

Expected a .debug_gdb_scripts section
Repository: electron
Issue: Accelerators should be localized in the menu
Labels: enhancement :sparkles:,component/menu

Right now, for an accelerator like `Ctrl+F`, Electron shows `Ctrl+F` in the menu while other Windows apps show `Strg+F` on my German system.

Electron shows `Ctrl+F`:
![image](https://cloud.githubusercontent.com/assets/2641501/7550387/ea543d1e-f660-11e4-96e5-7c5f783112b3.png)

Chrome shows `Strg+F`:
![image](https://cloud.githubusercontent.com/assets/2641501/7550388/f745c984-f660-11e4-80f4-4199397d4766.png)

There are multiple options:
1. Use the strings provided by Chrome. They are in `src/ui/strings/translations/ui_strings_LOCALE.xtb`.
2. Create our own strings, probably based on the Chrome strings. We don't need all the strings from Chrome, we just need [four](https://code.google.com/p/chromium/codesearch#chromium/src/ui/strings/translations/ui_strings_de.xtb&q=KEY_COMBO_NAME&sq=package:chromium&type=cs&l=35) (the 4 we need are marked, plus `Suche` which we don't need). Ok, maybe we also need a few more like `Space` = `Leerzeichen`, `Del` = `Entf`, `Insert` = `Einfg`, ...
3. Let the application handle what to display, so the application is responsible for all translation stuff.

On a side note, for the first two options, the application should be able to override the language (to me, it doesn't matter whether Electron auto-detects the language or not).
Repository: thefuck
Issue: Display issue on Git for Windows CLI
Labels: help wanted,windows

![image](https://cloud.githubusercontent.com/assets/2048645/7551370/f62a9f28-f653-11e4-951e-be906098784d.png)

My cli: Using git for windows 1.9.5.msysgit.0 (latest)
Python 3.4.3 on windows.

Also, the require_confirmation setting has been set to True in the current screenshot.