test503 for rshriram

Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [kyessenov]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kyessenov: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-simple

prow/istio-pilot-e2e.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-simple

prow/istio-pilot-e2e.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
55b3b0d719dcec39271cde4ab0306cc8be322e87
link
/test e2e-bookInfo

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
843f76bfdd934b2be81616ae02116c359f0dd4d0
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


build css and js files inside static folder

Hello i know it's not a bug report or a feature request but i need a help to understand how can i use an external CDN url with nuxt.
So if i edit nuxt.config.js with publicPath: ${process.env.CDN}/assets/, after build + start i can't get my js or css files from : https://www.myWebsite.com/assets/d54f5fde97r654.js, I think it's normal because i'm using CDN and i must upload theses files to my CDN. But i don't think that this is a good solution because my CDN upload automaticly resources from myWebsite.com, if '/assets/d54f5fde97r654.js' exist in myWebsite it will be uploaded to CDN with the same path automaticly.
I think if I can copy css and js files or all dest folder content to static folder during the build process, it will resolve my issue.
Or any suggestion?

This question is available on Nuxt community (#c9027)

I have same problem and solved by nuxt serverMiddleware to set a static folder
 serverMiddleware: [     { path: '/static-example', handler: serveStatic(__dirname + '/.nuxt/dist/client') }   ]


Use XZ to compress deb & rpm packages [APL-2626]

What does this PR do?
This PR enables XZ compression for our deb & rpm packages

Motivation
XZ can be used in parallel, which decrease the build times in itself, but it also compress much better than gzip.
More details can be found in https://datadoghq.atlassian.net/browse/APL-2626

Additional Notes
Requires these 2 PRs:
https://github.com/DataDog/datadog-agent-buildimages/pull/496
https://github.com/DataDog/omnibus-ruby/pull/163
I haven't specified the skip-qa label, although if the entire test suite passes I'd tend to think no additional QA is required.

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Automated testing will handle most of the QA (it's enough to test that the package can be installed).
To my knowledge, only debian 8 isn't covered there, so the QA'er will need to try and manually install a package that was compressed by xz.

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

To override the number of CPU Omnibus can use with the number of CPU provided by KUBERNETES_CPU_REQUEST, I did it there https://github.com/DataDog/datadog-agent/blob/main/omnibus/omnibus.rb#L17-L19
Maybe this modification should go there too, because it is the same for all the recipes, WDYT?

To override the number of CPU Omnibus can use with the number of CPU provided by KUBERNETES_CPU_REQUEST, I did it there https://github.com/DataDog/datadog-agent/blob/main/omnibus/omnibus.rb#L17-L19 Maybe this modification should go there too, because it is the same for all the recipes, WDYT?

We need to be in a packager scope to set these properties, so I'm not sure we can do so from there. We could probably assign a global and use if once in the packager scope but I don't think this really improves the code

/merge

/merge

/merge

/merge

/merge

/merge

/merge

/remove

/merge


Custom endpoint support

In our case, the Azure OpenAI API's are published to the Azure APIM API gateway so the only way to call the Azure openAI api's is to call a custom endpoint on a custom domain. To call the api's exposed on the api gateway we have to pass the following:
HTTP header - subscriber-application-key
User access Token (JWT token)
Note that this is not an onprem proxy, the Azure open AI api's are published on the API gateway to apply extra controls. We do not get Azure OpenAI key to pass. Only the subscriber application key

@rohitgabriel We have a document which describes how to use Semantic Kernel together with APIM, see: https://learn.microsoft.com/en-us/semantic-kernel/deploy/use-ai-apis-with-api-management
Have you seen this and are still having issues?

Thanks for the link, will let you know after testing

Following the document has been helpful, thanks, this can be closed

No more assistance needed

Updated document: https://jordanbeandev.com/how-to-set-up-an-asp-net-core-api-to-call-an-azure-openai-instance-that-is-behind-azure-api-management-and-requires-both-a-subscription-key-an-oauth2-access-token-using-semantic-kernel-c/


QuaggaJS in background (without displaying the camera)

Hello,
I want to use the library, and start the scanning, but don't wanna display the camera view (target), so what I did is that I removed the targert property from Quagga.Init method, and the camera was gone, is this a good/efficient solution? Or I need to seek for more configs? Thanks.

how would you have any idea what's happening, if you're not showing the user?  i'm not sure this would even work, because if there's no target, where is it acquiring the image from?

how would you have any idea what's happening, if you're not showing the user? i'm not sure this would even work, because if there's no target, where is it acquiring the image from?

The camera is placed and is stable, just put the barcode in front of it and the library scans it. Just like at a supermarket, does the cashier have a camera to scan the products? No, he just passes the items through the scanner.

yes, but a laser scanner doesn't require an image to operate, whereas quagga does require an image.  i mean, if it works like that, then i guess it confirms that the input continues to work even if there's no output.
i just can't imagine a situation where an image based scanner is good enough that you don't need/want to have the ability for the user to tell if the barcode is even visible to the scanner or not.

This is how it works: it asks for camera permissions, when you use the .start method, it starts scanning the camera in the background, and when it detects a barcode it calls "onDetected". Displaying the camera to the client is optional.

well, to answer your question, if it works, then that's probably the way to do it.  The other talk is just that I doubt you'll find it to be a good user experience, because image decoders are significantly pickier about what and how they'll read something than a laser scanner is, which just needs to have the entire barcode covered by the laser.  If you do find it workable, great :-)
Have a good one


Redirect to another location

By default sish will redirect to root location, like domain.tld/location
How can I redirect to another location when I type domain.tld to www.domain.tld?
Many thanks

Hi @thangnd85 ,
is this using the -r, --redirect-root-location string               The location to redirect requests to the root domain flag? If so, you can redirect this to anywhere as it will use any url for the redirect.

Hi @thangnd85 ,
is this using the -r, --redirect-root-location string               The location to redirect requests to the root domain flag? If so, you can redirect this to anywhere as it will use any url for the redirect.

Thanks for support.
But i tried, it redirect to child
Example my domain is: domain.com
And I redirect root location to www.domain.com
Then, when i type domain.com, it redirect to domain.com/www.domain.com instead of www.domain.com

Thanks for support.
But i tried, it redirect to child
Example my domain is: domain.com
And I redirect root location to www.domain.com
Then, when i type domain.com, it redirect to domain.com/www.domain.com instead of www.domain.com


ctlptl docker-desktop set cli.useCloudCli false sometimes fails

one of the teams we work with uses
ctlptl docker-desktop set cli.useCloudCli false

to automatically disable cloud cli in docker-desktop. But if you're using an old version of docker-desktop, this fails, because that setting does not exist.
we should probably have an --ignore-not-found for docker-desktop set commands

now think this was a PEBKAC, the invocation was:
ctlptl docker-desktop set cli.useCloudCli: false

just need to make the error message a little better


missing perspectiveTransform

Hi! This library seems to be missing perspectiveTransform. Is there a particular reason this might be? (Or maybe there is just more work needed to wrap a lot more functions?)
Thanks a bunch for your work! 😃

Hi! Well, yes, there's a reason; we decided not to wrap core module of OpenCV because it mainly contains non-CV stuff that is already available in Torch or at least can be easily done with existing functionality: optimization algorithms, linear system solvers, SVD, common math operations, algorithms on vectors and matrices etc.
In particular, perspectiveTransform can be implemented as follows:
function perspectiveTransform(src, M)
    -- number of channels
    local C = M:size(1) - 1

    local vecPrime = src * M:t()[{{1,C}, {}}]
    for i = 1,C+1 do
        vecPrime[{{}, i}]:add(M[{i, C+1}])
    end

    local wPrime = vecPrime[{{}, C+1}]
    wPrime:apply(function(x) return x == 0 and math.huge or x end)
    for i = 1,C do
        vecPrime[{{}, i}]:cdiv(wPrime)
    end

    return vecPrime[{{}, {1,3}}]
end

This code did not work for me in transforming a contour (an Nx1x2 IntTensor), which is what I usually use perspectiveTransform for.
I'm using this code. It works for transforming contours, but I'm sure does not work for all the things that OpenCV's perspectiveTransform works for. Also I'm a Torch novice so this is probably not the cleanest or most efficient implementation.
local function perspectiveTransform(src, M)
  src = src:double()
  local result = src:clone()

  for i = 1, src:size(1) do
    local v = torch.Tensor(1, 3)
    v[1][1] = src[i][1][1]
    v[1][2] = src[i][1][2]
    v[1][3] = 1
    local vprime = v * M:t()
    local z = vprime[1][3]
    if z == 0 then z = math.huge end
    result[i][1][1] = vprime[1][1] / vprime[1][3]
    result[i][1][2] = vprime[1][2] / vprime[1][3]
  end

  return result
end

Yes, my code is for Nx3 or Nx2 Tensor, so one should squeeze it first.


Ais/entrepreneurdocs

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[ ] You have registered new documents to /sidebars.js if any.

Just two super-small nits.
Massive thanks, looking forward to being able to share these things with the world!


How do I access role within session data?

I see in the access.ts file, the session appears to have a role property as part of it's data. I don't have that data available within my session.

Hey @wendtcode the session data is set in your keystone.ts file.
To add role you would do something like
const auth = createAuth({
  listKey: 'User',
  identityField: 'subjectId',
  sessionData: `id name email role { id name }`,
  autoCreate: true,

That is if role is a separate list, this is graphql so you can add anything that sits on the role list
Hope this helps.

Aha! Thanks for the suggestions!


Unsigned integer literals still showing up red in the IDE

I tried adding the plugin to Multiplatform Settings, and looking at code in the mingwX64 target from a macOS machine. Worked great for the most part, but was still seeing red code around unsigned integers. See screenshot for example

@russhwolf Do you have the same issue if reading the code in IDE in Windows?
@sellmair Do you know what could be the cause of this issue?

I do have a slight idea what might be going on. Maybe the native stdlib is not attached to the mingw source set.
My investigation can be tracked here:
https://youtrack.jetbrains.com/issue/KT-47124
👍

Can you please try if
    sourceSets.configureEach {
        languageSettings.apiVersion = "1.5"
        languageSettings.languageVersion = "1.5"
    }

Resolves your issue?

This is not a CompleteKotlin issue:
Root cause can be tracked here: https://youtrack.jetbrains.com/issue/KTIJ-18938
Workaround above should work ✅


Statusnook template

This template adds Statusnook, I am the maintainer.
GitHub repo: https://github.com/goksan/statusnook
I've tested statusnook.yaml with a Docker Compose deployment in the Coolify UI.

Thank you for the PR! Very nice app @goksan! Congrats!


komorebi: Add version 0.1.7

[x] I have read the Contributing Guide.

There is still a problem with this manifest, komorebi.exe does not work when launched from the shim, although it works when launched directly. I hope someone here can help with that.
Related issue: https://github.com/LGUG2Z/komorebi/issues/95

Can you try changing the text in the associated .shim file to use the actual version number, instead of "current", and see if that works?

Can you try changing the text in the associated .shim file to use the actual version number, instead of "current", and see if that works?

Doesn't work unfortunately

Hmm. I can think of one method, but it's really ugly - copy the 2 files into $shimdir during installation, and delete them during uninstallation.

Doesn't work unfortunately

For both komorebi.exe and komorebic.exe?
Hmm. In that case, I can think of one method, but it's really ugly - copy the 2 .exe files into $shimdir during installation, and delete them during uninstallation.

komorebic does work, the problem is with komorebi.exe not launching properly.

Hmm. In that case, I can think of one method, but it's really ugly - copy the 2 .exe files into $shimdir during installation, and delete them during uninstallation.

That's the solution in https://github.com/LGUG2Z/komorebi/issues/95#issuecomment-1022766335 and it does work (just replacing komorebi.exe)

What is the error on launching komorebi.exe when installed from the manifest in this PR?

What is the error on launching komorebi.exe when installed from the manifest in this PR?

Running komorebi --help produces an output:
komorebi 0.1.6
Jade Iqbal<EMAIL_ADDRESS>A tiling window manager for Windows

USAGE:
    komorebi.exe [OPTIONS]

OPTIONS:
        --ffm        Allow the use of komorebi's custom focus-follows-mouse implementation
    -h, --help       Print help information
    -V, --version    Print version information

But running just komorebi returns nothing, even if the rust log is set to full ($env:RUST_LOG = 'trace'; $env:RUST_BACKTRACE = 'full').

Pushed the workaround for the meanwhile.

Hey @rashil2000, I'm the maintainer of komorebi and I'm trying to figure out what is causing the behaviour described above. It seems like both binaries are compiled correctly and capable of execution, but the main event loop of komorebi is never entered into when it is executed as a shim.
The main event loop of komorebi has an arg check as a prerequisite which basically enforces that if the binary is called with only one arg (ie. komorebi.exe), or if it is called with two args, a positive value must have been set for the --ffm flag (this is negative by default if the flag is not passed).
The only thing that I can imagine would cause this behaviour is if the shim is adding additional unexpected arguments when the binary is called.
I'm happy to make changes upstream to avoid any hacks on the scoop manifest if you can point me in the right direction!

Thanks @rashil2000, I believe I have identified the issue.
The komorebi.exe in the shims folder starts a separate process with the same name which is not the 'real' komorebi.exe. This shim process remains running throughout the entire execution time of the real komorebi.exe process, which then causes [this check to ensure only one copy of the process is ever running at a time](https://github.com/LGUG2Z/komorebi/blob/e2ae9b1207102e1e303ea6d62f4439ac82fe91de/komorebi/src/main.rs#L344 to fail.
I will push a fix for this issue and cut a new release to allow the execution to continue if a shim process of the same name is running when this check is reached.

On a side note, this check typically logs an error (see the code block linked above) to the console when the binary is executed directly- I think that the shim has been swallowing this error which has made it harder for end users (and me!) to identify the root cause of this issue.

@sitiom https://github.com/LGUG2Z/komorebi/releases/tag/v0.1.8 should no longer have use the workaround with this release 🎉

Great, thanks for resolving the issue!

/verify


Commit boost API - Generate Proxy Key, Signing Request

PR Description
Commit boost API - Generate Proxy Key, Request Signature. Implementation of https://commit-boost.github.io/commit-boost-client/api

[x] Routes and Handler for Generate Proxy Key CommitBoostGenerateProxyKeyHandler, CommitBoostGenerateProxyKeyRoute
[x] Custom SSZ classes for calculating signing roots (BLSProxyDelegation, SECPProxyDelegation,ProxyDelegation)
[x] Create BLS and/or SECP proxy keys (utility classes ProxyKeyGenerator and SigningRootGenerator)
[x] Unit/Acceptance tests

Other changes:

Renamed  DepositSigningRootUtil to Web3SignerSigningRootUtil
Refactored  EthPublicKeyUtils to use Bouncycastle libraries to handle SECP Keys.
 However, added additional code to use actual GVR if Commit Boost client changes its implementation in future.
Updated Eth2Subcommand. Moved network overrides cli options to mixin class PicoCliNetworkOverrides
Updated Eth2Runner and register CommitBoostGenerateProxyKeyRoute
Updated SECP bulkloader to handle compressed public keys as identifiers
Updated DefaultArtifactProvider to add proxy signers. Refactored the code to make it a bit cleaner

Fixed Issue(s)

See #1017
Documentation

[x] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

Testing

[x] I thought about testing these changes in a realistic/non-local environment.

@jframe this monstrous PR is ready for review, give it a go at your convenience :)

@jframe this monstrous PR is ready for review, give it a go at your convenience :)

Is there anyway to break this PR into smaller PRs? It's quite big to review at the moment

Closing as PR was too long.


AggregateTestFixture should invoke methods annotated with @Autowired to inject dependecies into aggregate

Since axon framework requires empty constructor, i use method annotated @Autowired to inject dependencies by spring.
but in unit tests no spring contex is created so those methods are not called by axon test framework
or mayby there is a way to call such methods directly in the test execution phase?

The Aggregates and Sagas Axon constructs should, in essence, not use @Autowired methods or constructors at all.
For dependency injection into an Aggregate or Saga, you can simply add the dependency as a parameter to your message handling functions.
This would look a little bit like so:
@CommandHandler
public void handle(SomeCommand command, @Autowired MyDependency myDependency) {
    // ...
}

Axon uses so-called ParameterResolvers for this. For more on these, check out this page of the Reference Guide.
I will close this issue, as the usage of setter injection is not recommended at the moment.
Hoping to have clarified the situation and what your workaround can be, @radoslawsobies!


python311Packages.pywaze: 0.3.0 -> 0.4.0

Diff: https://github.com/eifinger/pywaze/compare/refs/tags/v0.3.0...v0.4.0
Changelog: https://github.com/eifinger/pywaze/releases/tag/v0.4.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 254290 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.waze_travel_time
python310Packages.pywaze
python310Packages.pywaze.dist
python311Packages.pywaze
python311Packages.pywaze.dist


(Hedgehog) Rangers Message Table Support

[ ] Implement a way to save this format.

Threw this out as I didn't like it at all.


[Ruby] Use nil as default value for all types

The constructors were setting an empty string or the number zero depending on the type of the attribute. This made it impossible to tell if the value was actively set to zero or if it was not set at all.
Now it's possible to omit the stockLimitation attribute for unlimited stock.
This change also reduces the JSON file size by omitting these values.

Closes #13

@mkllnk The import feature has been merged and this should be tested with it, right?

The import feature has been merged and this should be tested with it, right?

Right. I tested and found that setting arrays to nil broke an assumption of the importer. So I'm keeping the empty array for now.
It would be even better if the importer knew if a property is a collection instead of checking for an array value. But for now this is good enough and we can improve it when we have a use case for it.
This has been rebased on the main branch.

@lecoqlibre This has been waiting for two months now. Is there any concern over merging this?


Syntax error when launching bjarkan snap

alan@hal:~$ snap install bjarkan
bjarkan 1.2.0 from 'willdeberry' installed
alan@hal:~$ bjarkan
/snap/bjarkan/24/usr/bin/python3: 1: /snap/bjarkan/24/usr/bin/python3: Syntax error: word unexpected (expecting ")")

Also tested the version in the edge channel and had the same issue.

Thanks for the update! Was not aware of this issue.


Movie details on clicking cards

There's an option for searching a movie, and then movie details appear on clicking that button by fetching from omdb api.
There are many cards on the homepage, but on clicking those, it redirects nowhere. I will make it redirect it to a page where all the movie details from omdb appears.

Hey @0mBudsMan i am interested to work on this project can you assign me this issue ?

Hey @QAZIMAAZARSHAD i am interested to work on this project can you assign me this issue ?

@0mBudsMan is currently working on this issue, I have assigned you other issues.


When compiling with Version Xcode 15.4, an error occurs and cannot be compiled

When compiling with Version 15.4, an error occurs and cannot be compiled：Command PhaseScriptExecution failed with a nonzero exit code；Could you fix it?

I'm not seeing a failure myself, and CI is successfully building (mostly) with Xcode 15.4. Can you include the Xcode build output?

Could not launch “Chime”
Domain: IDELaunchErrorDomain
Code: 20
Recovery Suggestion: Runningboard has returned error 5. Please check the system logs for the underlying cause of the error.
User Info: {
DVTErrorCreationDateKey = "2024-07-09 14:03:11 +0000";
DVTRadarComponentKey = 968756;
IDERunOperationFailingWorker = IDELaunchServicesLauncher;
}
Could not launch “Chime”
Domain: IDELaunchErrorDomain
Code: 20
Recovery Suggestion: Runningboard has returned error 5. Please check the system logs for the underlying cause of the error.
User Info: {
DVTRadarComponentKey = 968756;
IDERunOperationFailingWorker = IDELaunchServicesLauncher;
}
The operation couldn’t be completed. Launch failed.
Domain: RBSRequestErrorDomain
Code: 5
Failure Reason: Launch failed.
Launchd job spawn failed
Domain: NSPOSIXErrorDomain
Code: 153
Event Metadata: com.apple.dt.IDERunOperationWorkerFinished : {
"device_model" = "MacBookPro18,1";
"device_osBuild" = "14.5 (23F79)";
"device_platform" = "com.apple.platform.macosx";
"dvt_coredevice_version" = "355.28";
"dvt_mobiledevice_version" = "1643.120.2";
"launchSession_schemeCommand" = Run;
"launchSession_state" = 1;
"launchSession_targetArch" = arm64;
"operation_duration_ms" = 61;
"operation_errorCode" = 20;
"operation_errorDomain" = IDELaunchErrorDomain;
"operation_errorWorker" = IDELaunchServicesLauncher;
"operation_name" = IDERunOperationWorkerGroup;
"param_debugger_attachToExtensions" = 0;
"param_debugger_attachToXPC" = 1;
"param_debugger_type" = 3;
"param_destination_isProxy" = 0;
"param_destination_platform" = "com.apple.platform.macosx";
"param_diag_MainThreadChecker_stopOnIssue" = 0;
"param_diag_MallocStackLogging_enableDuringAttach" = 0;
"param_diag_MallocStackLogging_enableForXPC" = 1;
"param_diag_allowLocationSimulation" = 1;
"param_diag_checker_tpc_enable" = 1;
"param_diag_gpu_frameCapture_enable" = 0;
"param_diag_gpu_shaderValidation_enable" = 0;
"param_diag_gpu_validation_enable" = 0;
"param_diag_memoryGraphOnResourceException" = 0;
"param_diag_queueDebugging_enable" = 1;
"param_diag_runtimeProfile_generate" = 0;
"param_diag_sanitizer_asan_enable" = 0;
"param_diag_sanitizer_tsan_enable" = 0;
"param_diag_sanitizer_tsan_stopOnIssue" = 0;
"param_diag_sanitizer_ubsan_stopOnIssue" = 0;
"param_diag_showNonLocalizedStrings" = 0;
"param_diag_viewDebugging_enabled" = 1;
"param_diag_viewDebugging_insertDylibOnLaunch" = 1;
"param_install_style" = 0;
"param_launcher_UID" = 2;
"param_launcher_allowDeviceSensorReplayData" = 0;
"param_launcher_kind" = 0;
"param_launcher_style" = 99;
"param_launcher_substyle" = 8192;
"param_runnable_appExtensionHostRunMode" = 0;
"param_runnable_productType" = "com.apple.product-type.application";
"param_structuredConsoleMode" = 1;
"param_testing_launchedForTesting" = 0;
"param_testing_suppressSimulatorApp" = 0;
"param_testing_usingCLI" = 0;
"sdk_canonicalName" = "macosx14.5";
"sdk_osVersion" = "14.5";
"sdk_variant" = macos;
}
System Information
macOS Version 14.5 (Build 23F79)
Xcode 15.4 (22622) (Build 15F31d)

https://github.com/ChimeHQ/Chime/assets/34976742/5df0b165-2a7f-4934-b499-4520e89de362

Ok, the clue we have here is the wording "PhaseScriptExecution failed" in that video. I think one of the build phases is failing, possibly a custom one. Can you check out the build logs from the failing part here:

I also just tried a fresh check out, and followed all the published directions. The build succeeded for me locally.

clone the repo
git submodule update --init --recursive
cp User.xcconfig.template User.xcconfig
update User.xcconfig with your personal information

I just followed the above operation steps and re-operated, but there was still a build error

Showing Recent Issues
/Users/ronghuahe/Library/Developer/Xcode/DerivedData/Edit-crkitrsxwzqthlfurzlqmhjrafsc/Build/Intermediates.noindex/Edit.build/Debug/Edit.build/Script-C9E878122A9F563F0018340C.sh: line 2: /Users/ronghuahe/myIOS/iOS: No such file or directory
Showing Recent Issues
Command PhaseScriptExecution failed with a nonzero exit code

Ok we're getting closer.
"/Users/ronghuahe/myIOS/iOS: No such file or directory"
This path does not appear in the project, so it must be coming from the environment somehow. I bet what is happening is the Edit target's "validate build" step is failing. This relies on being able to execute a shell script on your machine within the Xcode build phases.
Somewhere in that same screen is a smaller red x next to the failing step. If you can find it and select it, you can expand it to see the full output of the step.
Also, just to test. What happens if you run this from your shell?
/bin/sh --version

Ah, that's the detail I needed. This is indeed a bug! I can reproduce the failure with a fresh check out in a path containing spaces. Looking into that now.
For your (very resonable) follow-up questions, the answer is that this open source version is not functional at the moment. I'm working on it! Right now, the Quick Look extension does work for the languages currently supported. And I'm working right now on expanding that to match the released version #20 .

Thanks so much both for reporting this and for having the patience to help me debug. I was able to fix up the build phase and this now will work when the repo is at a path with spaces.


Not working

MissingPluginException(No implementation found for method share on channel com.juanito21.simpleShare/share)

Error:
Attribute provider#androidx.core.content.FileProvider@authorities value=(com.sandm.flutter_app.adv_provider) from [:simple_share] AndroidManifest.xml:10:13-64
is also present at [:open_file] AndroidManifest.xml:14:13-64 value=(com.sandm.flutter_app.fileProvider).
Suggestion: add 'tools:replace="android:authorities"' to  element at AndroidManifest.xml:8:9-16:20 to override.
C:\Users\snehil\AndroidStudioProjects\lib_genesis\flutter_app\android\app\src\debug\AndroidManifest.xml:15:17-68 Error:
Attribute meta-data#android.support.FILE_PROVIDER_PATHS@resource value=(@xml/simple_share_provider_paths) from [:simple_share] AndroidManifest.xml:15:17-68
is also present at [:open_file] AndroidManifest.xml:20:17-50 value=(@xml/filepaths).
Suggestion: add 'tools:replace="android:resource"' to  element at AndroidManifest.xml to override.


Missing Policy in Windows 10 22H2 environment

[Enter feedback here]
Hi
You describe a new policy.
I can't find this policy in our Windows 10 22H2 environment.
We can I get this policy template (admx, adml)?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 0e5e70e7-fd3f-bbb3-9c24-9f2552f94d6d
Version Independent ID: 0e5e70e7-fd3f-bbb3-9c24-9f2552f94d6d
Content: Use Windows Update for Business and Windows Server Update Services (WSUS) together - Windows Deployment
Content Source: windows/deployment/update/wufb-wsus.md
Product: windows-client
Technology: itpro-updates
GitHub Login: @mestew
Microsoft Alias: mstewart

Thanks @CanastraRF for taking the time to share this with the Windows documentation team. I’m sorry that you’re having problems, and I want to make sure it gets to the right people that can help. There are a couple of options available to provide the best level of support for this issue:

Post to Windows on Q&A. These are moderated by community members who are very experienced, knowledgeable, and helpful.
Use Microsoft Support to search for the issue, or open a case with professional support.
Contact your Microsoft Unified Support team, such as a Customer Engineer or Customer Success Account Manager.

I'm closing this issue as there's nothing actionable for docs at this time. The policy on my Win11 device says that same as in the doc, that it's available on Windows 10, version 2004 and above. And we don't publish the ADMX files as far as I know. You can look at the CSP documentation, which has more details on the policies: https://learn.microsoft.com/windows/client-management/mdm/policy-csp-update#setpolicydrivenupdatesourcefordriverupdates


Treat pseudo selectors like emotion / styled-components?

The currently implemented way to emit a block with a qualifier according to https://github.com/futursolo/stylist-rs/blob/c59568c943deb0500397915f4d462ac5aa140b03/packages/stylist-core/src/ast.rs#L127-L137 is to prefix that with the injected class name. I believe an erroneous additional whitespace was being added already in css-in-rust. To show a few examples:
.extra { background: red; }
:not(body) { background: red; }
&:not(body) { background: red; }

becomes
.stylist-1 .extra { background: red; }
.stylist-2 :not(body) { background: red; }
//        ^ erroneous space, this is now a descendant combinator!
.stylist-3:not(body) { background: red; }
//        ^ all good!

I would propose a more radical change while fixing the above: simply do not inject the class when no '&' is present in the condition. Just replace all '&' and leave the rest be in blocks. So the above would become
.extra { background: red; }
// Now a global style
:not(body) { background: red; }
//  A breaking change but actually consistent with what material-ui does :)
.stylist-3:not(body) { background: red; }
//        ^ all good!

I believe the proposed syntax handling is more consistent with the handling in material-ui where that '&' seems to be required or at least commonly used, see for example https://github.com/mui-org/material-ui/blob/49b75b08a6584c930fdd23a0eb1fe28f6e1ffce5/packages/material-ui/src/Button/Button.js#L229-L241
This would also readdress #4 and give an escape-hatch for components that for some reason can't specify a class for an element, e.g. because they use a third-party js library that render the components.

As CSS-In-Rust cannot parse your example, I have modified it to the example below:
background: red;
:not(body) { background: red; }
&:not(body) { background: red; }
.extra { background: red; }

Under master branch, css-in-rust generates the following stylesheet:
.CustomComponent-4597592370089325376 {
background: red;
}
.CustomComponent-4597592370089325376 :not(body)  {
                                    /* space here */
background: red;
}
 .CustomComponent-4597592370089325376:not(body)  {
background: red;
}
.CustomComponent-4597592370089325376 .extra  {
background: red;
}

Stylist generated stylesheet:
.stylist-Rh02X9kD {
background: red;
}
.stylist-Rh02X9kD :not(body) {
background: red;
}
.stylist-Rh02X9kD:not(body) {
background: red;
}
.stylist-Rh02X9kD .extra {
background: red;
}

The behaviour matches.
However, this behaviour do not match emotion or styled-components.
They generated something like:
.hYXQpd{
background:red;
}
.hYXQpd:not(body){
background:red;
}
.hYXQpd:not(body){
background:red;
}
.hYXQpd .extra{
background:red;
}

In addition, material-ui v5 is going to adopt emotion and drop their own css solution.
So this will also become the behaviour of material-ui.
So the problem becomes whether to treat pseudo selectors like styled-components or emotion.
As for global styles, I think the proper way to deal with it is like emotion and styled-components, introduce a <Global /> component for global styles and otherwise all scoped.
IMO, the main advantage of a css-in-rust / css-in-js solution over the traditional CSS / SCSS / LESS solution is to have scoped css. Do not require it opens a door for people to write global css everywhere (sometimes without noticing it).
Which really defeats the purpose of such a solution.
In short, You should use <GlobalStyle /> to apply global styles, or add your class name to <html /> manually.

Thanks for doing the digging! I concur that it should match emotion behavior where documented and intended. When reading through their docs, they always use the ampersand. If only there was documentation available with strict rules instead of example after example. I don't want to just match their output if that might be coincidental and not intended.
E.g: I don't understand
.hYXQpd .extra{
    /* ^ why the extra space here but not for pseudo selectors? */
    /* seems to fall into the same trap of now being a descendant combinator */
background:red;
}

I'd rather fail or at least warn about potentially misleading output than exactly match some codified but not formulated spec.
Apart from that, remaining issue then for me is only that extra space, completely on your side with Global!

I found https://github.com/emotion-js/emotion/issues/235#issuecomment-320438237 that claims that the & is a requirement from v7 onwards, contradicting observed behavior...


Input array error when loading contextily basemap

Discovered by one of my students, there looks to be a new bug when using contextily to add background maps to plots.
"could not broadcast input array from shape (512,512,4) into shape (256,256,4)"
A similar issue has been reported here

https://gitlab.com/deltares/imod/imod-python/-/issues/567

The same error is evident in the geopandas gallery.

https://geopandas.org/en/latest/gallery/plotting_basemap_background.html

I was wondering if anyone had any thoughts as to the cause (OpenStreetMap?)

Thanks for the report! See https://github.com/geopandas/contextily/issues/223 for the analysis and solutions.


Missing some important list blocks

List blocks that I definitely would like to see because they are empirically useful:

repeating
shuffled
sorted
reversed
++
first

I still need a (++) block, which seems more complex than the ones I've built so far.


Merge from esri master20170910

Merge changes for IEDA geoportal from development branch and tag

merge and delete the development branch.


Monzo

Do you have any plans to update this to work with Mondo's new name, Monzo?

I'll accept pull requests to change and update the name, but I don't have the time to update it myself.


Can we blur the background outside a MaterialDialog when it pops up?

Can we blur the background outside a MaterialDialog when it pops up?

I intend to leave default Android dialog behavior.


Code for EEF MVC tutorials

Merging sample projects first so individual PRs for EF MVC tutorials can be staged and snippets will be accessible to them.

cc @Rick-Anderson


Hide preview plugin starts background process, but never completes, doesn't hide previews.

What system are you running Yazi on?
Linux X11 i3wm
What terminal are you running Yazi in?
kitty 0.37.0
yazi --debug output
Yazi
    Version: 0.3.3 (Arch Linux 2024-09-05)
    Debug  : false
    OS     : linux-x86_64 (unix)

Ya
    Version: 0.3.3 (Arch Linux 2024-09-05)

Emulator
    Emulator.via_env: ("xterm-kitty", "")
    Emulator.via_csi: Ok(Kitty)
    Emulator.detect : Kitty

Adapter
    Adapter.matches: Kitty

Desktop
    XDG_SESSION_TYPE           : Some("x11")
    WAYLAND_DISPLAY            : None
    DISPLAY                    : Some(":1")
    SWAYSOCK                   : None
    HYPRLAND_INSTANCE_SIGNATURE: None
    WAYFIRE_SOCKET             : None

SSH
    shared.in_ssh_connection: false

WSL
    WSL: false

Variables
    SHELL              : Some("/bin/zsh")
    EDITOR             : Some("/usr/bin/nvim")
    VISUAL             : None
    YAZI_FILE_ONE      : None
    YAZI_CONFIG_HOME   : None

Text Opener
    default: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block  : Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

Multiplexers
    TMUX               : false
    tmux version       : tmux 3.5a
    ZELLIJ_SESSION_NAME: None
    Zellij version     : No such file or directory (os error 2)

Dependencies
    file             : 5.46
    ueberzugpp       : No such file or directory (os error 2)
    ffmpegthumbnailer: 2.2.3
    magick           : 7.1.1-41
    fzf              : 0.56.3
    fd               : 10.2.0
    rg               : 14.1.1
    chafa            : No such file or directory (os error 2)
    zoxide           : 0.9.6
    7z               : 17.05
    7zz              : 24.09
    jq               : 1.7.1

--------------------------------------------------

Describe the bug
I have installed the hide-preview plugin as per the instructions, and added
on   = "#"
run  = "plugin hide-preview"
desc = "Hide or show preview"

to my keymap.conf
When I press the appropriate key, the preview is not hidden, but a process indicator appears in the yazi status bar at the bottom, and when I quit yazi I get a notification about processes running.

Minimal reproducer
To reproduce:
install hide preview plugin
ya pack -a yazi-rs/plugins:hide-preview
add keymap to ~/.config/yazi/keymap.toml
on   = "#"
run  = "plugin hide-preview"
desc = "Hide or show preview"

hit # (or whatever key you've set up, I tried a few in case it was because of a conflict).
expected: previews are hidden, like it says on the tin
observed: previews not hidden, background process notifications that never complete. Nothing in log file.
Anything else?
No response

Version: 0.3.3 (Arch Linux 2024-09-05)

Please make sure you're on the latest nightly, or at least the latest stable version 0.4.1
https://github.com/yazi-rs/plugins/blob/62f078905b4de55f19e328452c8a1f889ff2f6f4/README.md?plain=1#L1-L4

https://github.com/sxyazi/yazi/releases/tag/nightly

Thanks. I'd just installed it through pacman.


Add #1209: logout button in navbar

Description
This PR adds a button to the navigation bar that allows the user to log out of the application in only one click from any page. A button has also been added to the mobile nav bar for when the user is on a mobile device. When the Disable Auth option is enabled, the logout button does not show in the navbar.
Closes #1209
Type of change
Please delete any options that are not relevant.

User interface (UI)
New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[ ] My code needed automated testing. I have added them (this is optional task) Not required

Screenshots (if any)
Desktop navbar:

Mobile navbar:

Fixed merge conflicts

Showing logout button in the header is not so common in modern web design, but I agree the logout button is hard to find currently.
I plan to implement dropdown menu like this, so we could put the logout button there.

Yes, I know what you mean about it not being so common as it would normally be in a drop-down like you showed. Let me know if you need any help implementing that.

Suddenly interested in doing some css, I put this pr to 1.15.0 milestone.
TODO:

[ ] Display real username
[ ] Get the first letter of username and uppercase


Bug: Re-Announcing Lobby

When opening a game, the announcer says "Join at  etc" and all the name announcements. If the host goes into game settings, and returns to the lobby, those announcements are read again.
Potentially, surface up a property for the <Speak> component named something like hasBeenSpoken that will prevent any announcement from being spoken more than once.

This is a non-issue now. The only thing the host can do is start the game or end the game.
If they refresh, the announcements may all happen again. I don't think there's a way to prevent that; or at least not worth preventing it.


Add more description to journal_file field

The issue is that journal_file init option purpose may be confusing for some users. For example, they may think they need to specify a file for logging, like in journalctl.
Only after hours of debugging your code I was able to find out that it expects me to specify path to .beancount file.

Somehow related to #48.

Latest version should not require setting the journal file and just operate on the current file by default
My next focus is documentation and testing when I can find time to work on it

How is this currently working? I've pointed the journal file to my main.bean file, which imports various files (accounts, commodities, etc) and the journals for each year. Currently, this is throwing an error, and the workaround is importing the accounts, etc to the journal too.

the journal file should parse that file and any include statements in each file parsed
could you provide more detail or a small example to  help debug this issue?


set-exit-code doesn't doesn't validate new exit code values

The set-exit-code fn in the exit-code ns should validate new-exit-code before swapping the value in the private atom.

This might be done between lines 24 and 25 of that ns via a condition such as (if (integer? new-exit-code)

Done, TY!


Update skeleton dependencies patch and some feature releases #31

@dd137 - in regards to your comment that this package.json is only used for the skeleton app. Can confirm that however starting my app with the debug option shows that the electronAppScaffold is overwriting any changes and reverts to the original values:
VERBOSE  meteorApp:  updating .meteor/.gitignorez
VERBOSE  meteorApp:  desktopHCP is not enabled, removing required packages
VERBOSE  electronAppScaffold:  clearing desktop-build
VERBOSE  electronAppScaffold:  creating /Users/andreaswest/Documents/workspace/frontend/.meteor/desktop-build/app
VERBOSE  electronAppScaffold:  copying skeleton app
VERBOSE  electronAppScaffold:  writing package.json
VERBOSE  electronApp:  updating package.json fields
DEBUG  electronApp:  writing updated package.json
INFO  electronApp:  updating list of package.json's dependencies
DEBUG  electronApp:  merging settings.json[dependencies]
DEBUG  electronApp:  merging settings.json[plugins]
DEBUG  electronApp:  merging dependencies from modules
DEBUG  electronApp:  writing updated package.json
VERBOSE  electronApp:  calculating compatibility version
DEBUG  electronApp:  meteor-desktop compatibility version is  3
DEBUG  electronApp:  .desktop compatibility version is  3
INFO  electronApp:  issuing node_modules install from electron-builder

Integration test fails here:
=> Errors while parsing arguments:
While adding package<EMAIL_ADDRESS>error: no such version<EMAIL_ADDRESS>While adding package<EMAIL_ADDRESS>error: no such version<EMAIL_ADDRESS>ERROR  electronApp:  error while checking for required packages:  Error: Error: adding packages failed
at MeteorApp.ensureDesktopHCPPackages (/home/runner/work/meteor-desktop/meteor-desktop/lib/meteorApp.js:83:23)
at processTicksAndRejections (internal/process/task_queues.js:95:5)
at ElectronApp.build (/home/runner/work/meteor-desktop/meteor-desktop/lib/electronApp.js:316:13)
at MeteorDesktop.build (/home/runner/work/meteor-desktop/meteor-desktop/lib/index.js:131:9)
at Context. (/home/runner/work/meteor-desktop/meteor-desktop/tests/integration/integration.test.js:239:13)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test-integration: mocha -t 10000 -r @meteorjs/reify "tests/+(integration)/**/*.test.js"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test-integration script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/runner/.npm/_logs/2023-10-04T15_35_13_451Z-debug.log
Error: Process completed with exit code 1.
I'm not running HCP as it was causing errors with my app, so no idea what's wrong here.

Ah, I just noticed some of the same problems (https://github.com/Meteor-Community-Packages/meteor-desktop/pull/13#issuecomment-1748450031).
@a4xrbj1 I see. Looks indeed like package.json is rewritten every time. That explain you initial observation. Let's update the dependencies in skeletonDependencies.js, as per this PR and your issue 👍


Available in Windows 10, 64-bit system?

Been trying to install sknn into a new virtual environment I created on my Win 10, 64 bit machine.
conda install scikit-neuralnetwork
initially I get the error message:
Fetching package metadata .........
Solving package specifications: .
PackageNotFoundError: Package not found: '' Package missing in current win-64 channels:

scikit-neuralnetwork

Does sknn not exist for Win 64?

I don't think it's available on conda. I had to use pip to install sknn. (On mac)


Migrate to use  <amp-story-auto-analytics>

Feature Description

Going forward we might want to exclusively use <amp-story-auto-analytics>.
Perhaps we want to use it in conjunction with amp-analytics during a migration period.
Considerations
What about Site Kit? Site Kit currently prints an <amp-analytics> tag on stories. Should they also switch? Needs coordination.
Alternatives Considered

Additional Context

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Pinging @felixarntz for awareness. We might want to update Site Kit to print <amp-story-auto-analytics> instead of <amp-story-analytics>.

@swissspidy Please keep me in the loop on whether this should be done or not. I've created a tracking issue on Site Kit side. What I'm mostly curious about is whether that new component has any drawbacks in using - for example the amp-analytics component allows for more flexibility in the events setup, even though it is missing the useful defaults that the new one provides.

@swissspidy Please keep me in the loop on whether this should be done or not. I've created a tracking issue on Site Kit side. What I'm mostly curious about is whether that new component has any drawbacks in using - for example the amp-analytics component allows for more flexibility in the events setup, even though it is missing the useful defaults that the new one provides.

One would not be able to customize the events anymore, but with auto analytics there's a guaranteed set of events configured out of the box. Which makes it easier to develop stories-specific analytics dashboards and the like, as we 100% what data is being collected.

One would not be able to customize the events anymore, but with auto analytics there's a guaranteed set of events configured out of the box. Which makes it easier to develop stories-specific analytics dashboards and the like, as we 100% what data is being collected.

Hmm that sounds like a trade-off I'm not sure about what's better for Site Kit. I wonder whether it would be possible for the amp-story-auto-analytics component to still support an (optional) JSON configuration for additional configuration and events. As it currently stands, I don't see enough of a benefit from what I know so far.

Suggestion by @choumx in sprint planning meeting just now:
Use amp-story-auto-analytics for new users freshly installing the plugin or setting up GA for the first time, and keep pre-existing users on the "old" amp-analytics configuration if they have GA set up already.
For pre-existing users, show a note on the settings page about them using the "old" configuration including documentation on how to migrate to the new configuration (location TBD) and offer CTA to switch to the new way (UX TBD).
Needs some UX and clarity on where that documentation resides, what it should cover, who writes it, etc.

@aaskedall flagging that the UX for this is needed for next Monday sprint start.

"An improved version of analytics is now available. Learn how to migrate to our recommended version. "
Added some additional copy and an alert badge. Can the user make this change right inline or do they have to make it elsewhere and this settings view will update to reflect that?

Can the user make this change right inline or do they have to make it elsewhere and this settings view will update to reflect that?

Yeah that could all be done inline. As simple as clicking on a "Migrate" button that saves their choice.

For copy I'd suggest:

[ Migrate ] An improved analytics configuration is now available (learn more)."

Avoids the word "version" which may be conflated with version of Google Analytics itself.

I created #9045 to implement this behind a feature flag while UX is being finalized.
@aaskedall Could you perhaps provide an updated mock that takes into account our feedback above? There should be some button to perform that migration. Clicking the button could simply display a success snackbar message.

Here is an updated mock. Just need to some language for the snackbar. How instant is this change or is it a comeback in a few minutes?

Just need to some language for the snackbar.

I'd have gone with something like "Successfully migrated analytics configuration" or "Settings saved"

How instant is this change or is it a comeback in a few minutes?

It's instant

@aaskedall Oh, btw, we already have some text below the "Google Analytics" label. So the migration text would need to be displayed below.

Also, perhaps it should be a secondary button and not a primary one?

@aaskedall @choumx See #9050 for a demo of how it currently works.

#9050 is merged now. PTAL again at the demo there.
Documentation is the biggest remaining point besides design.


Compatibility with node 0.10.x

My CI don't want to go green on 0.10.x due to client-request. Problem is due to require('util').debuglog which has been introduced by 0.11.x
There is a "polyfill" node-debuglog but adding a dependency for this thing would be really shame.
I don't known why we even tests our code on 0.10 ^^ Maybe it would be great to write it in doc.
Just for your knowledge... I will sleep on it and probably remove this version.

Quick fix when you don't want to drop 0.10 support:
debuglog = require('debuglog')
util = require('util')
util.debuglog = debuglog

Just saw this! Fixed by making debuglog a noop on 0.10 vs adding a dependency.
Thanks!


☂The Infrastructure is reconciled constantly when a Shoot is hibernated and terraform apply pod returns error

What happened:
After triggering hibernation the terraform pod is spawn, although it is not needed. If error occurs during execution of this pod this will continue each 30 seconds. During this infinite loop a logs will be created which will load the fluentd buffer with no end point messages, because elastic-search pod is scaled to 0.
What you expected to happen:
No terraform should be triggered unless Infrastructure is marked for deletion.
How to reproduce it (as minimally and precisely as possible):
Make your own shoot in Gardener WebUI with valid credentials.
Hibernate the shoot.
Change the credentials in cloudprovider secret with fake ones.
Annotate the Infrastructure with gardener.cloud/operation=reconcile
Anything else we need to know?:
Environment:

Gardener version: 1.3.1
Kubernetes version (use kubectl version): 1.15.11
Cloud provider or hardware configuration: azure
Others:

The deployInfrastructure task is only triggered once during maintenance, so you suggest to not trigger it at all if shoot is hibernated?
Can be done, however, for existing shoots/infrastructures of hibernated clusters there is no way to stop it (other than letting the .status.lastOperation being manually patched by an administrator).

Can we just put this cone snipped
if cluster.Shoot.Spec.Hibernation != nil &&        cluster.Shoot.Spec.Hibernation.Enabled != nil &&       *cluster.Shoot.Spec.Hibernation.Enabled {	 return reconcile.Result{}, nil	 }
Into the reconciliation function of the Infrastructure controller?

This would a) require every infrastructure controller to change and b) leave the Infrastructure objects in a stale state.

This issue does not require immediately reaction so after time every infrastructure controller will change eventually.
To prevent Infrastructure objects being in a stale state we could change the LastOperationState to Aborted with desription "Abort Reconciliation because Shoot is hibernated" .

The deployInfrastructure task is only triggered once during maintenance, so you suggest to not trigger it at all if shoot is hibernated?

Wasn't the problem in this case that the TF pod failed, again and again (unauthorised or something)? The pod was recreated every 30 seconds or so, always creating logs, always pushing into fluentd that could never ship its logs out and was at max capacity all the time. I think we have a timeout of 24h or so for an operation, but then again, in 24h the next maintenance is triggered.
It would be already a great help if we get that feature out for new clusters/woken up clusters and have a back-off for the TF pod in general. As it is right now, we burn through resources and ship endless logs that suffocate fluentd, which means we might miss other valuable logs along the way.
Uups, I see too late, the ticket was already closed. What about the TF back-off question above @rfranzke and @vpnachev?

Yes, that's another improvement in the Infrastructure controllers of the various extensions that are using Terraform. I'll look into it as part of the other planned improvements (e.g., today, we sometimes don't display the actual error messages because of non-optimal timeouts).

Excellent, thanks. Is there a ticket to refer from here, because this one is closed, but the subject "Infrastructure is reconciled constantly when a Shoot is hibernated and terraform apply pod returns error" makes sense.

No, I'll reopen this issue and refer to it from the various extension repos.
/reopen

Because we want our logs back, please:
/priority/critical

/in-progress
/quality

/remove status in-progress
/close
as all related PRs have been merged and will be part of the next extension releases


HHH-15211 Fix Embeddable element collection converter bug

https://hibernate.atlassian.net/browse/HHH-15211
The root cause is we did not sync up 'equalness' with 'hash code' for such edge case (collection of Embeddable element to be converted)

LGTM

Thanks @NathanQingyangXu :+1:


Docker container increasing size by 1gig everyday

I have installed mongodb and meteor on an Amazon Instance EC2. my main challenge is that the container is increasing in size everyday and am running in short of disk space. I really need your assistance here, am running in short of disk space now.

@wattletimber I gave a suggestion on https://github.com/docker/docker/issues/7758#issuecomment-223905181, but I see you posted the same question in 10 different locations; please note that the GitHub issue tracker is for tracking bugs and feature requests, not a support forum, for questions about using docker, please use either one of:

forums.docker.com
the #docker IRC channel on freenode
StackOverflow

I'll close this issue because this is a support question, not a bug


Use Github Actions

It still needs to be resolved how passwords will look

I've added encrypted secrets at the org level, available to this repo, same as used in the travis build: BINTRAY_USERNAME, BINTRAY_PASSWORD, SONATYPE_USERNAME, SONATYPE_PASSWORD, and DEPLOY_DOCS_TOKEN. These need to be injected into the environment with something like:
env:
  BINTRAY_PASSWORD: ${{ secrets.BINTRAY_PASSWORD }}


Ft 318 Create make command for debugging tests

Which Issue was this PR made for?

Closes #318

What is within the scope with this PR
Adding make test-debug
Looks like this

What is NOT in the scope with this PR
Related Issue
Other sidenotes that reviewer should be aware of

AUTOMATED: Deployed to https://pr-319.dev.mochatto.com


Reset promiseRef on useQuery hook

Shrugsy reported this in Discord, and this solves it, but I haven't considered any other implications yet. Will review again later.

Is there anything I can do about hot reloading causing queries to get unsubscribed and become 'uninitialized'? On a fresh CRA install, after adding a small rtk query setup, the behaviour I observe is like so:

Launch app (npm start)
Open console
Open redux devtools
[in devtools + console] Observe that data fetches normally (query fulfils in devtools, data & statuses populate in console)
[in ide] Edit and save this file (e.g. edit the logging text) to trigger a hot reload
[in devtools] Observe that the 'pokemonApi/subscriptions/unsubscribeResult' action is dispatched after the hot reload
[in console] Wait 60 seconds (or length of 'keepUnusedDataFor' option)
[in devtools] Observe that the 'pokemonApi/queries/removeQueryResult' action is dispatched
[in console] Observe that 'isUninitialized' is now back to true
A repo that reproduces this is here: https://github.com/Shrugsy/simple-rtk-app
Editing this file will allow you to reproduce it with the above steps: https://github.com/Shrugsy/simple-rtk-app/blob/master/src/features/pokemon/PokemonItem.jsx

I'm not 100% sure this is the "most correct solution".
From what I gather, this is caused by the cleanup callback being run as if the component was unmounted. But after that, the other effect does not act like the component was re-mounted.
This is because we never clean the promiseRef in the cleanup callback and then compare it with a shallowly memoized value and see "no change".
https://github.com/rtk-incubator/rtk-query/blob/next/src/react-hooks/buildHooks.ts#L192
So I think a better fix would be to reset the refs in the cleanup callback in
https://github.com/rtk-incubator/rtk-query/blob/next/src/react-hooks/buildHooks.ts#L216-L218

I'm not 100% sure this is the "most correct solution".
From what I gather, this is caused by the cleanup callback being run as if the component was unmounted. But after that, the other effect does not act like the component was re-mounted.
This is because we never clean the promiseRef in the cleanup callback and then compare it with a shallowly memoized value and see "no change".
https://github.com/rtk-incubator/rtk-query/blob/next/src/react-hooks/buildHooks.ts#L192
So I think a better fix would be to reset the refs in the cleanup callback in
https://github.com/rtk-incubator/rtk-query/blob/next/src/react-hooks/buildHooks.ts#L216-L218

I agree. I think the one counterpoint though is that if someone dispatched updateSubscriptionOptions after scheduling a removal in a non-hook implementation, the original solution in middleware would be a 'safety net'. Then again, middleware probably shouldn't be concerned with that? :shrug:

@msutkowski it is a valid concern, but I think we should catch that case over in https://github.com/rtk-incubator/rtk-query/blob/ef4208904d46fbe4fbbf505e0e8a78d5670e149b/src/core/buildInitiate.ts#L125-L127 and either do nothing or even throw an exception?


get_iplayer: 3.33 -> 3.34

Description of changes
https://github.com/get-iplayer/get_iplayer/wiki/release330to339#release334

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

builds and runs, and fixes the bug mentioned in the changelog.
Result of nixpkgs-review pr 265660 run on x86_64-linux 1

2 packages built:

get_iplayer
get_iplayer.man


override sessionDidReceiveChallenge method to bypass server trust issue in Alamofire5

Hi all we were using alamofire 4.9.1 before and recently i have upgraded to Alamofire 5 and with that we are facing bellow error
The certificate for this server is invalid. You might be connecting to a server that is pretending to be “XXX.XXX.XX.X” which could put your confidential information at risk." UserInfo={NSLocalizedRecoverySuggestion=Would you like to connect to the server anyway?,
In Alamofire 4.9.1, we had resolve same issue by override didReceive challenge method as bellow
delegate.sessionDidReceiveChallenge = { session, challenge in
               var disposition: URLSession.AuthChallengeDisposition = .performDefaultHandling
               var credential: URLCredential?

               if challenge.protectionSpace.authenticationMethod == NSURLAuthenticationMethodServerTrust {
                   disposition = URLSession.AuthChallengeDisposition.useCredential
                   credential = URLCredential(trust: challenge.protectionSpace.serverTrust!)
               } else {
                   if challenge.previousFailureCount > 0 {
                       disposition = .cancelAuthenticationChallenge
                   } else {
                       credential = WebAPIManager.manager.session.configuration.urlCredentialStorage?.defaultCredential(for: challenge.protectionSpace)
                       if credential != nil {
                           disposition = .useCredential
                       }
                   }
               }
               return (disposition, credential)
           }

This is essentially equivalent to using the DisabledTrustEvaluator, but that's a bad idea. Like the code you posted, it accepts any server trust as valid, even those from other domain or unexpected certificates, so I would suggest not using it. Instead, if your production server has an invalid certificate, you should fix the certificate if you can. If you can't do that, you should, instead of accepting all trusts, which is a security issue, pin the server's certificate directly. That way you at can at least be sure you're connecting to the right website with the right certificate.


Xcode test plan for Swift package?

Is it possible? It'd be much nicer than the current XCTSkip/If/Unless solution.

Xcode schemes can select a subset of tests.


Generation of Tasks

Generation tasks
This is can be a subproject on its own due to its complexity
Questionnaire response to a task generation

Who, When and how

Generation task offline
-Generation of tasks offline on the Android client

Related and being tracked in issue #15


Add image format option

Adds a --format option.  Value is supplied to the image save function so that you can specify PNG/JPEG/whatever.  Defaults to JPEG if not specified.

Great idea! I've added this functionality to both the backend and frontend in https://github.com/saharmor/dalle-playground/commit/cad1753193ed15c06c6dddc076d0f5d4a04653a7


Show red outline on invalid subscribers/members input pill

Show red outline on invalid subscribers/members input pill and disable create/add button wherever applicable. For the disabling button part, manual testing would be better, attached screenshots for the red outline:

Screenshots and screen captures:
Create channel add subscribers screen:

Edit channel add subscribers screen:

Create group members screen:

Edit group members screen:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Playing with the UI, I think it would be better not to disable the Add button, and have it get disabled/enabled all the time as you type.
I think it would work well to just leave it enabled, but trigger the red outline and shaking animation if you click the button while there is some bad input in the field.

The channel/group creation UI change looks good to me.

I've made the requested changes, haven't updated the screenshots, do have a look if you get some time, otherwise, I'll polish it more if needed tmr!

Works for me in manual testing!

Overall this looks good. Tiniest nit: Change instances in comments of "clicking enter" to "pressing Enter"--given that that's a keyboard thing.


Chinese translation update 1.18

Some of the new keys don't yet have a Chinese translation, so those names may be subject to change, but merging is OK since maybe there won't be anyone to translate them in near future.
Still working on translating Regions Unexplored, their translations are about to come.


Improvement to take source configuration directly from source contain…

Improvement to take source configuration directly from source container's

DB_ environment variables, which will be easier to maintain with container
itself.

added

- DB_DESTINATION for destination type
- DB_DESTPATH    for destination_directory_name
- DB_VOLUMES     for volume dirs to take backup
- DB_BACKUP_PRECMD="mysqldump all > /data/dump.sql"
  for cmd to run using exec before taking backup
- DB_BACKUP_POSTCMD="echo done"
  for cmd to run using exec after taking backup
- DB_RESTORE_PRECMD="echo run restore"
  for cmd to run using exec before restoring
- DB_RESTORE_POSTCMD="echo done"
  for cmd to run using exec after restoring

Hello Sharad! Your PR seemed nice! Any reason you closed it?

Hi,
In hurry few changes were not gone into this commit, so I have closed it.
e.g. forget to add app/worker-env.sh
I was trying to ensure that it is complete and useful, and will not create
another bug
new changes are in https://github.com/sharad/docker-backup/
I am still trying to make proper commit, once I am sure it is correct
by tonigh or tomorrow, may amend commit, then I will again
raise pull request with complete changes
Thanks for consideration, and creating useful software.

--
/sharad
On Thu, Jun 15, 2017 at 12:54 PM, Arnaud de Mouhy<EMAIL_ADDRESS>wrote:

Hello Sharad! Your PR seemed nice! Any reason you closed it?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/akerbis/docker-backup/pull/9#issuecomment-308651099,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGvA3EACjxgHUaOYF69c2UPsREYf3Vgks5sENw2gaJpZM4N5FXF
.

Hi Arnaud,
Please see if pull request
https://github.com/akerbis/docker-backup/pull/10/commits
is useful with docker-backup
Thanks,
/sharad
On Thu, Jun 15, 2017 at 3:08 PM, Sharad Pratap<EMAIL_ADDRESS>wrote:

Hi,
In hurry few changes were not gone into this commit, so I have closed it.
e.g. forget to add app/worker-env.sh
I was trying to ensure that it is complete and useful, and will not create
another bug
new changes are in https://github.com/sharad/docker-backup/
I am still trying to make proper commit, once I am sure it is correct
by tonigh or tomorrow, may amend commit, then I will again
raise pull request with complete changes
Thanks for consideration, and creating useful software.

--
/sharad
On Thu, Jun 15, 2017 at 12:54 PM, Arnaud de Mouhy <
<EMAIL_ADDRESS>wrote:

Hello Sharad! Your PR seemed nice! Any reason you closed it?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/akerbis/docker-backup/pull/9#issuecomment-308651099,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGvA3EACjxgHUaOYF69c2UPsREYf3Vgks5sENw2gaJpZM4N5FXF
.


cannot accept/deny signups in TF

The buttons aren't visible. Is this because signup is not enabled for this app?

Most likely. Enabling signup will make it visible in the list to everyone though, so I'm not sure what the best solution is. Another boolean 'show in list' or something?


Breaking Change: strcharpart too many arguments

Hey, I'm loving the plugin.
There was a breaking change with the latest commit related to strcharpart
E5108: Error executing lua: Vim:E118: Too many arguments for function: strcharpart
stack traceback:
	[C]: in function 'strcharpart'
	.../.local/share/nvim/lazy/leap.nvim/lua/leap/highlight.lua:67: in function 'highlight-cursor'
	.../.local/share/nvim/lazy/leap.nvim/lua/leap/main.lua:730: in function 'get_first_pattern_input'
	.../.local/share/nvim/lazy/leap.nvim/lua/leap/main.lua:987: in function 'leap'
	.../.local/share/nvim/lazy/leap.nvim/plugin/init.lua:53: in function <.../.local/share/nv

Environment: PopOS
NeoVim Version:
❯ nv --version
NVIM v0.10.0-dev-133+gdbcd1985d
Build type: Release
LuaJIT 2.1.0-beta3

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/share/nvim"

Run :checkhealth for more info

Package Manager: LazyNvim

Got it. My bad; thanks for the quick reply. Updating now...


the genesis state root is incorrect (state reward must be -1)

Apparently there is a difference in waht genesis is used in generated blockchain tests:
Error: Importing raw RLP block, block was expected to be valid! (if it was intended, check that it is not in Valid blocks test suite) Error importing raw rlp block: ToolChainManager:: unknown parent hash 0x01e7fb1a4d848f80871bfea58f87511cc1cf8a09afaaeab66fd35281aeb76491

I reject the block because genesis block hash is different. The genesis blockhash is important as it contains the pre state. if you calculate your block on a different genesis we have disagreement.
To compute genesis blockhash I take what is needed for env file from this info
        "genesisBlockHeader" : {
            "bloom" : "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
            "coinbase" : "0x0000000000000000000000000000000000000000",
            "difficulty" : "0x020000",
            "extraData" : "0x00",
            "gasLimit" : "0x016345785d8a0000",
            "gasUsed" : "0x00",
            "hash" : "0x01e7fb1a4d848f80871bfea58f87511cc1cf8a09afaaeab66fd35281aeb76491",
            "mixHash" : "0x0000000000000000000000000000000000000000000000000000000000000000",
            "nonce" : "0x0000000000000000",
            "number" : "0x00",
            "parentHash" : "0x0000000000000000000000000000000000000000000000000000000000000000",
            "receiptTrie" : "0x56e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421",
            "stateRoot" : "0x9d539b791e8d942b88859bc2954a489e2102b00b7fd7106c4f769e0e860596fd",
            "timestamp" : "0x00",
            "transactionsTrie" : "0x56e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421",
            "uncleHash" : "0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347"
        },

feed it to the tool to calculate stateRoot from given pre and then calculate the blockhash

the issue seems to be that stateRoot in the test blockheader (genesis) is different to the one t8n return when I compute pre state stateroot

Fixed by #177


Module install/build fails on Heroku

Install works fine locally (OSX), though I had to change some permissions to allow the binary to be written outside the node_modules directory.
Why not just write the binary downloaded from S3 to node_modules/nodegit?
Fetching repository, done.
Counting objects: 108, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (92/92), done.
Writing objects: 100% (95/95), 12.12 KiB | 0 bytes/s, done.
Total 95 (delta 66), reused 5 (delta 2)

-----> Node.js app detected
-----> Resetting git environment
-----> Requested node range:  ^0.10.28
-----> Resolved node version: 0.10.33
-----> Downloading and installing node
-----> Restoring node_modules directory from cache
-----> Pruning cached dependencies not specified in package.json
-----> Exporting config vars to environment
-----> Installing dependencies
       npm WARN package.json<EMAIL_ADDRESS>No repository field.
       npm WARN package.json<EMAIL_ADDRESS>No README data
       npm WARN package.json<EMAIL_ADDRESS>No repository field.

       ><EMAIL_ADDRESS>install /tmp/build_1cd14a456bfa97e314e7aa8c1a34a6c7/node_modules/nodegit
       > node install

       [nodegit] Fetching binary from S3.
       [nodegit] Failed to install prebuilt, attempting compile.
       [nodegit] Determining dependencies.
       [nodegit] Detecting vendor/libgit2.
       [nodegit] Removing outdated vendor/libgit2.
       [nodegit] Detecting vendor/libssh2.
       [nodegit] Removing outdated vendor/libssh2.
       [nodegit] Detecting vendor/http_parser.
       [nodegit] Removing outdated vendor/http_parser.
       [nodegit] Fetching vendor/libgit2.
       [nodegit] Fetching vendor/libssh2.
       [nodegit] Fetching vendor/http_parser.
       [nodegit] Building native node module.
       [nodegit] Failed to build and install nodegit.
       Command failed: ../vendor/libssh2/src/openssl.c: In function '_libssh2_pub_priv_keyfile':
       ../vendor/libssh2/src/openssl.c:767: warning: value computed is not used
       ../vendor/libssh2/src/transport.c: In function '_libssh2_transport_send':
       ../vendor/libssh2/src/transport.c:716: warning: suggest braces around empty body in an 'if' statement
       cc1plus: error: unrecognized command line option "-std=c++11"
       make: *** [Release/obj.target/nodegit/src/nodegit.o] Error 1
       gyp ERR! build error
       gyp ERR! stack Error: `make` failed with exit code: 2
       gyp ERR! stack     at ChildProcess.onExit (/tmp/build_1cd14a456bfa97e314e7aa8c1a34a6c7/node_modules/nodegit/node_modules/node-gyp/lib/build.js:267:23)
       gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
       gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:810:12)
       gyp ERR! System Linux 3.8.11-ec2
       gyp ERR! command "node" "/tmp/build_1cd14a456bfa97e314e7aa8c1a34a6c7/node_modules/nodegit/node_modules/.bin/node-gyp" "clean" "configure" "build" "--python" "/usr/local/bin/python2"
       gyp ERR! cwd /tmp/build_1cd14a456bfa97e314e7aa8c1a34a6c7/node_modules/nodegit
       gyp ERR! node -v v0.10.33
       gyp ERR! node-gyp -v v1.0.2
       gyp ERR! not ok

We'll need this now for v0.2.5

The issue with the unrecognized command line option is a problem with make. I know it doesnt work with make 4.6, and does with 4.8, not sure about 4.7.

Is this still an issue for you?

I'm closing this. Let me know if you still have an issue once you've tried a newer g++.