{"action":"created","issue":{"url":"https://api.github.com/repos/wspr/fontspec/issues/292","repository_url":"https://api.github.com/repos/wspr/fontspec","labels_url":"https://api.github.com/repos/wspr/fontspec/issues/292/labels{/name}","comments_url":"https://api.github.com/repos/wspr/fontspec/issues/292/comments","events_url":"https://api.github.com/repos/wspr/fontspec/issues/292/events","html_url":"https://github.com/wspr/fontspec/issues/292","id":230907644,"number":292,"title":"Errors while defining extra NFSS variants for a family","user":{"login":"jmfarrow","id":119180,"avatar_url":"https://avatars3.githubusercontent.com/u/119180?v=3","gravatar_id":"","url":"https://api.github.com/users/jmfarrow","html_url":"https://github.com/jmfarrow","followers_url":"https://api.github.com/users/jmfarrow/followers","following_url":"https://api.github.com/users/jmfarrow/following{/other_user}","gists_url":"https://api.github.com/users/jmfarrow/gists{/gist_id}","starred_url":"https://api.github.com/users/jmfarrow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmfarrow/subscriptions","organizations_url":"https://api.github.com/users/jmfarrow/orgs","repos_url":"https://api.github.com/users/jmfarrow/repos","events_url":"https://api.github.com/users/jmfarrow/events{/privacy}","received_events_url":"https://api.github.com/users/jmfarrow/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-05-24T03:05:59Z","updated_at":"2017-05-26T02:36:55Z","closed_at":null,"body":"The following\r\n```\r\n\\setmainfont{CMU Serif}[\r\n  Ligatures=TeX,\r\n  BoldFont={CMU Serif Extra},\r\n  FontFace={bc}{n}{Font = CMUSerif-BoldNonextended, FakeStretch=0.8}\r\n]\r\n```\r\nthrows an unknown-key for Letters.\r\n\r\nUsing `FontFace={bc}{n}{CMUSerif-BoldNonextended}` but then obviously no stretching takes place.\r\n\r\nA reminder from stackexchange:\r\n\r\n> Looks like a bug to me. Probably the default setting of Letters=Smallcaps shouldn't be used when processing this font. Report it at github.com/wspr/fontspec/issues – Ulrike Fischer May 17 at 8:17\r\n\r\nand, indeed commenting out the attempt to make small caps:\r\n\r\n```\r\n          \\clist_put_left:Nn \\l__fontspec_fontfeat_curr_clist {Letters=SmallCaps}\r\n```\r\n\r\nstops the error.\r\n\r\nI'd make a patch so it only tries this under the right circumstances but my familiarity with both fontspec and the LaTeX3 interfaces aren't up to the task."},"comment":{"url":"https://api.github.com/repos/wspr/fontspec/issues/comments/304175572","html_url":"https://github.com/wspr/fontspec/issues/292#issuecomment-304175572","issue_url":"https://api.github.com/repos/wspr/fontspec/issues/292","id":304175572,"user":{"login":"wspr","id":8169,"avatar_url":"https://avatars3.githubusercontent.com/u/8169?v=3","gravatar_id":"","url":"https://api.github.com/users/wspr","html_url":"https://github.com/wspr","followers_url":"https://api.github.com/users/wspr/followers","following_url":"https://api.github.com/users/wspr/following{/other_user}","gists_url":"https://api.github.com/users/wspr/gists{/gist_id}","starred_url":"https://api.github.com/users/wspr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wspr/subscriptions","organizations_url":"https://api.github.com/users/wspr/orgs","repos_url":"https://api.github.com/users/wspr/repos","events_url":"https://api.github.com/users/wspr/events{/privacy}","received_events_url":"https://api.github.com/users/wspr/received_events","type":"User","site_admin":false},"created_at":"2017-05-26T02:36:55Z","updated_at":"2017-05-26T02:36:55Z","body":"@jmfarrow — thanks for the detailed investigations. While the expl3 code can be improved here, there's probably also an argument that fontspec could/should do things differently. I'll try to take a look some time… "}}
{"actor":{"display_login":"wspr"}}