Build Question

I am having issues compiling strait from the examples in the the README.
I am running this in a VM on VirtualBox.  I am running Ubuntu 16.04.2 LTS with all updates.  I am running AMQP-CPP version 2.7 built from source.
Below is my source code, something simple just to get compilation at this point, but I am getting an undefined reference in TcpConnection.  The compilation output is below the source.
After compiling, I have run ldconfig, and the output of 'ldconfig -p' | grep amqp' shows libamqpcpp.so
`
#include <amqpcpp.h>
#include
struct MyTcpHandler : public AMQP::TcpHandler
{
virtual void onConnected(AMQP::TcpConnection *connection)
{
std::cout << "MyTcpHandler: " << FUNCTION << std::endl;
}
virtual void onError(AMQP::TcpConnection *connection, const char *message)
{
    std::cout << "MyTcpHandler: " << __FUNCTION__ << std::endl;
    std::cout << std::string(message) << std::endl;
}

virtual void onClosed(AMQP::TcpConnection *connection)
{
    std::cout << "MyTcpHandler: " << __FUNCTION__ << std::endl;
}

virtual void monitor(AMQP::TcpConnection *connection, int fd, int flags)
{
    std::cout << "MyTcpHandler: " << __FUNCTION__ << std::endl;
}

};
int main()
{
std::cout << "Hello from " << FUNCTION << std::endl;
// create an instance of your own tcp handler
MyTcpHandler myHandler;

AMQP::Address address("amqp://localhost/");

// create a AMQP connection object
AMQP::TcpConnection connection(&myHandler, address);

// and create a channel
//AMQP::TcpChannel channel(&connection);

// use the channel object to call the AMQP method you like
//channel.declareExchange("", AMQP::fanout);
//channel.declareQueue("hello");
//channel.bindQueue("", "hello", "hello");

return 0;

}
`
mkdir -p build
g++ -Wall -Wno-deprecated-declarations -g -std=c++14 -pthread -I/usr/include -L/usr/lib -lamqpcpp -Isrc -c src/run.cpp -o build/run.o
g++ -Wall -Wno-deprecated-declarations -g -std=c++14 -pthread -I/usr/include -L/usr/lib -lamqpcpp -Isrc -o run.out build/run.o
build/run.o: In function main': /home/work/Devel/playground/rabbitmq/amqp-cpp-learning/src/run.cpp:38: undefined reference to AMQP::TcpConnection::TcpConnection(AMQP::TcpHandler*, AMQP::Address const&)'
build/run.o: In function AMQP::Connection::~Connection()': /usr/include/amqpcpp/connection.h:70: undefined reference to AMQP::ConnectionImpl::~ConnectionImpl()'
build/run.o: In function AMQP::TcpConnection::~TcpConnection()': /usr/include/amqpcpp/tcpconnection.h:118: undefined reference to vtable for AMQP::TcpConnection'
/usr/include/amqpcpp/tcpconnection.h:118: undefined reference to vtable for AMQP::TcpConnection' /usr/include/amqpcpp/tcpconnection.h:118: undefined reference to AMQP::Watchable::~Watchable()'
collect2: error: ld returned 1 exit status
Makefile:19: recipe for target 'run.out' failed
make: *** [run.out] Error 1

Same problem ...

Same problem ...

I got the same problem... I'm pretty sure the library is not built properly. I can see that the TcpConnection class is inside the src directory. Did anyone manage to resolve this?

I cannot reproduce this problem. Please provide the exact steps that you took all the way from downloading AMQP up to running your test.
I copied your code to the file "test.cpp" in my home directory, and fixed some small small issues that were probably caused by the github editor (like underscores that were missing). I then did the following things to install AMQP-CPP, compile your example program and get it up and running:
git clone git@github.com:CopernicaMarketingSoftware/AMQP-CPP.git
cd AMQP-CPP
make
sudo make install
cd ~
g++ -std=c++11 test.cpp -lamqpcpp -lpthread
./a.out

I have seen this when a project is mixing libraries that do/don't use position independent code (gcc's -fPIC) command.
In cmake, this can be resolved using:
cmake -DCMAKE_BUILD_TYPE=DEBUG -DCMAKE_POSITION_INDEPENDENT_CODE=ON <path_to_root>
This is only coming from experience; I've not seen anything in the log to indicate this is the problem.

@EmielBruijntjes I had the same issue. Apparently I solve the issue by compiling the release package of the library located here https://github.com/CopernicaMarketingSoftware/AMQP-CPP/releases .
Cloning the repo gave me the same result as @mrberman87 , could you please review that issue?
Thanks

I am still waiting on the exact styeps to reproduce this issue.

@gchaviaras, can you try change 'FUNCTION ' to 'func' and let us know if it then works?

Cant reproduce right now, I will try to deploy a fresh install

I am still waiting for the exact steps to reproduce this issue.

Hi,
Am pretty new to windows and RabbitMQ. Sorry for the lengthy write-up, it's an attempt to explain my build issue clearly. Thanks for reading.
I am developing a AMQP client in C++. For that I thought I will use AMQP-CPP library and start developing the client. I took the latest repo via "Download Zip" option in the git hub. I am able to build the library successfully on Windows using cmake. This is my cmake output.
C:\GIT\AMQP-CPP-master>mkdir build & cd build
C:\GIT\AMQP-CPP-master\build>cmake ..
-- Building for: Visual Studio 14 2015
-- Selecting Windows SDK version  to target Windows 10.0.16299.
-- The C compiler identification is MSVC 19.0.24215.1
-- The CXX compiler identification is MSVC 19.0.24215.1
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: C:/GIT/AMQP-CPP-master/build
C:\GIT\AMQP-CPP-master\build>
I have created a default Win32 console application in VS 2015. And my main CPP file looks like this
#include "stdafx.h"
#ifdef NOMINMAX
#include <windows.h>
#else
#define NOMINMAX
#include <windows.h>
#undef NOMINMAX
#endif
#include <amqpcpp.h>
int main()
{
return 0;
}
When I try to build, am seeing this issues and my solution is not building fine.
1>------ Build started: Project: amqp_cpp_client1, Configuration: Debug Win32 ------
1>  stdafx.cpp
1>  amqp_cpp_client1.cpp
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(100): warning C4005: 'AF_IPX': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(452): note: see previous definition of 'AF_IPX'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(140): warning C4005: 'AF_MAX': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(471): note: see previous definition of 'AF_MAX'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(177): warning C4005: 'SO_DONTLINGER': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(394): note: see previous definition of 'SO_DONTLINGER'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(221): error C2011: 'sockaddr': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(1002): note: see declaration of 'sockaddr'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(421): error C2059: syntax error: 'constant'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(421): error C3805: 'constant': unexpected token, expected either '}' or a ','
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(556): warning C4005: 'IN_CLASSA': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(279): note: see previous definition of 'IN_CLASSA'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(562): warning C4005: 'IN_CLASSB': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(285): note: see previous definition of 'IN_CLASSB'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(568): warning C4005: 'IN_CLASSC': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(291): note: see previous definition of 'IN_CLASSC'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(579): warning C4005: 'INADDR_ANY': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(296): note: see previous definition of 'INADDR_ANY'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(581): warning C4005: 'INADDR_BROADCAST': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(298): note: see previous definition of 'INADDR_BROADCAST'
1>c:\program files (x86)\windows kits\8.1\include\shared\ws2def.h(615): error C2011: 'sockaddr_in': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(1006): note: see declaration of 'sockaddr_in'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(136): error C2011: 'fd_set': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(1014): note: see declaration of 'fd_set'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(156): warning C4005: 'FD_CLR': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(89): note: see previous definition of 'FD_CLR'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(171): warning C4005: 'FD_SET': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(94): note: see previous definition of 'FD_SET'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(180): error C2011: 'timeval': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(1030): note: see declaration of 'timeval'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(236): error C2011: 'hostent': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(1018): note: see declaration of 'hostent'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(249): error C2011: 'netent': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(172): note: see declaration of 'netent'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(256): error C2011: 'servent': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(1022): note: see declaration of 'servent'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(268): error C2011: 'protoent': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(1026): note: see declaration of 'protoent'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(364): error C2011: 'WSAData': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(314): note: see declaration of 'WSAData'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(462): error C2011: 'sockproto': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(486): note: see declaration of 'sockproto'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(504): error C2011: 'linger': 'struct' type redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(1010): note: see declaration of 'linger'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(517): warning C4005: 'SOMAXCONN': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(536): note: see previous definition of 'SOMAXCONN'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(551): warning C4005: 'FD_READ': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(554): note: see previous definition of 'FD_READ'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(554): warning C4005: 'FD_WRITE': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(555): note: see previous definition of 'FD_WRITE'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(557): warning C4005: 'FD_OOB': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(556): note: see previous definition of 'FD_OOB'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(560): warning C4005: 'FD_ACCEPT': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(557): note: see previous definition of 'FD_ACCEPT'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(563): warning C4005: 'FD_CONNECT': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(558): note: see previous definition of 'FD_CONNECT'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(566): warning C4005: 'FD_CLOSE': macro redefinition
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(559): note: see previous definition of 'FD_CLOSE'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1646): error C2375: 'accept': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(734): note: see declaration of 'accept'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1668): error C2375: 'bind': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(739): note: see declaration of 'bind'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1689): error C2375: 'closesocket': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(744): note: see declaration of 'closesocket'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1706): error C2375: 'connect': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(746): note: see declaration of 'connect'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1727): error C2375: 'ioctlsocket': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(751): note: see declaration of 'ioctlsocket'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1748): error C2375: 'getpeername': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(756): note: see declaration of 'getpeername'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1769): error C2375: 'getsockname': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(761): note: see declaration of 'getsockname'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1790): error C2375: 'getsockopt': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(766): note: see declaration of 'getsockopt'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1815): error C2375: 'htonl': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(773): note: see declaration of 'htonl'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1832): error C2375: 'htons': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(775): note: see declaration of 'htons'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1850): error C2375: 'inet_addr': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(777): note: see declaration of 'inet_addr'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1868): error C2375: 'inet_ntoa': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(779): note: see declaration of 'inet_ntoa'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1968): error C2375: 'listen': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(781): note: see declaration of 'listen'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(1987): error C2375: 'ntohl': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(785): note: see declaration of 'ntohl'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2004): error C2375: 'ntohs': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(787): note: see declaration of 'ntohs'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2021): error C2375: 'recv': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(789): note: see declaration of 'recv'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2044): error C2375: 'recvfrom': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(795): note: see declaration of 'recvfrom'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2071): error C2375: 'select': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(803): note: see declaration of 'select'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2096): error C2375: 'send': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(810): note: see declaration of 'send'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2119): error C2375: 'sendto': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(816): note: see declaration of 'sendto'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2146): error C2375: 'setsockopt': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(824): note: see declaration of 'setsockopt'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2171): error C2375: 'shutdown': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(831): note: see declaration of 'shutdown'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2191): error C2375: 'socket': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(835): note: see declaration of 'socket'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2216): error C2375: 'gethostbyaddr': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(842): note: see declaration of 'gethostbyaddr'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2238): error C2375: 'gethostbyname': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(847): note: see declaration of 'gethostbyname'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2255): error C2375: 'gethostname': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(849): note: see declaration of 'gethostname'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2295): error C2375: 'getservbyport': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(853): note: see declaration of 'getservbyport'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2314): error C2375: 'getservbyname': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(857): note: see declaration of 'getservbyname'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2333): error C2375: 'getprotobynumber': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(861): note: see declaration of 'getprotobynumber'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2350): error C2375: 'getprotobyname': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(863): note: see declaration of 'getprotobyname'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2370): error C2375: 'WSAStartup': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(867): note: see declaration of 'WSAStartup'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2390): error C2375: 'WSACleanup': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(871): note: see declaration of 'WSACleanup'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2407): error C2375: 'WSASetLastError': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(873): note: see declaration of 'WSASetLastError'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2424): error C2375: 'WSAGetLastError': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(875): note: see declaration of 'WSAGetLastError'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2445): error C2375: 'WSAIsBlocking': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(877): note: see declaration of 'WSAIsBlocking'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2463): error C2375: 'WSAUnhookBlockingHook': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(879): note: see declaration of 'WSAUnhookBlockingHook'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2481): error C2375: 'WSASetBlockingHook': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(881): note: see declaration of 'WSASetBlockingHook'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2499): error C2375: 'WSACancelBlockingCall': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(883): note: see declaration of 'WSACancelBlockingCall'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2517): error C2375: 'WSAAsyncGetServByName': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(885): note: see declaration of 'WSAAsyncGetServByName'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2545): error C2375: 'WSAAsyncGetServByPort': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(893): note: see declaration of 'WSAAsyncGetServByPort'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2573): error C2375: 'WSAAsyncGetProtoByName': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(901): note: see declaration of 'WSAAsyncGetProtoByName'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2599): error C2375: 'WSAAsyncGetProtoByNumber': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(908): note: see declaration of 'WSAAsyncGetProtoByNumber'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2625): error C2375: 'WSAAsyncGetHostByName': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(915): note: see declaration of 'WSAAsyncGetHostByName'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2651): error C2375: 'WSAAsyncGetHostByAddr': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(922): note: see declaration of 'WSAAsyncGetHostByAddr'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2681): error C2375: 'WSACancelAsyncRequest': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(931): note: see declaration of 'WSACancelAsyncRequest'
1>c:\program files (x86)\windows kits\8.1\include\um\winsock2.h(2699): error C2375: 'WSAAsyncSelect': redefinition; different linkage
1>  c:\program files (x86)\windows kits\8.1\include\um\winsock.h(933): note: see declaration of 'WSAAsyncSelect'
1>c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(80): error C2440: 'initializing': cannot convert from 'u_short (__stdcall *)(u_short)' to 'uint16_t'
1>  c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(80): note: There is no context in which this conversion is possible
1>c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(93): error C2440: 'initializing': cannot convert from 'u_long (__stdcall *)(u_long)' to 'uint32_t'
1>  c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(93): note: There is no context in which this conversion is possible
1>c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(106): error C2568: '==': unable to resolve function overload
1>  c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(106): note: could be 'u_long htonl(u_long)'
1>c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(106): error C2568: '|': unable to resolve function overload
1>  c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(106): note: could be 'u_long htonl(u_long)'
1>c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(129): error C2440: 'initializing': cannot convert from 'u_short (__stdcall *)(u_short)' to 'int16_t'
1>  c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(129): note: There is no context in which this conversion is possible
1>c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(142): error C2440: 'initializing': cannot convert from 'u_long (__stdcall *)(u_long)' to 'int32_t'
1>  c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(142): note: There is no context in which this conversion is possible
1>c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(155): error C2568: '==': unable to resolve function overload
1>  c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(155): note: could be 'u_long htonl(u_long)'
1>c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(155): error C2568: '|': unable to resolve function overload
1>  c:\git\amqp-cpp-master\include\amqpcpp\outbuffer.h(155): note: could be 'u_long htonl(u_long)'
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
Please help me how to write a c++ client program using the AMQP-CPP library.

I suspect some of what you are seeing is caused by this.  I almost feel that this should be in a separate ticket because up until this point all failed builds have been Linux based. I'll let admins use their discretion :)

@zerodefect 👍  Adding #define _WINSOCKAPI_   resolved the errors and my application built fine. thank you.

just got exactly the same issue as @mrberman87 but successfully resolved that. My problems was I build AMQP-CPP with cmake but I didn't installed libssl-dev.
I installed that lib and built with make and issue gone.

Got exactly the same problem as mrberman87.
kubuntu 16.04
cmake version 3.5.1
make version GNU Make 4.1
gcc (Ubuntu 5.4.0-6ubuntu1~16.04.10) 5.4.0 20160609
amqpcpp versions master and tag v3.2.0 (did not check older versions)
This is a strange behavior of linker and add_sources macro. I fixed that with usage of cmake aux_source_directory command instead of add_sources macro.
patch.txt

This is an old issue, but I ran into it yesterday and wanted to leave a comment in case someone encounters the same problem.
In my case, I was trying to statically link libamqpcpp.a on Ubuntu into a shared object that will be dynamically loaded by MySQL as a plugin. The root problem is that shared libraries require position independent code but the libamqpcpp.a binary is not built this way by default. The errors generated while building the shared object were quite vague which made diagnosing the issue difficult. The solution is what zerodefect mentioned above: pass the -DCMAKE_POSITION_INDEPENDENT_CODE=ON flag to cmake when building the library.

I did not even know that it was possible to create a *.so file based on *.a files. What flags and arguments do you pass to the compiler for that?

This is the first time I've had to statically link into a shared object on Linux so it was a learning experience. The plugin is simple and is being manually built like so:
g++ -Wall -fPIC -c RabbitHandler.cpp -o RabbitHandler.o
g++ -Wall -fPIC -z defs -shared -DMYSQL_DYNAMIC_PLUGIN -I/usr/include/mysql \
  opsnotify.cc RabbitHandler.o /usr/local/lib/libamqpcpp.a -pthread -ldl \
  -o libopsnotify.so


1.0.0.alpha2 errors / Contract

Just started to play around 1.0.0.alpha2 and I am getting the following error and I am not sure if it's on my end something I am missing or it's indeed a bug:
class API::V2::AppointmentContract < Dry::Validation::Contract
  RSVP ||= ["going", "not_going"].freeze
  TYPE ||= %w(sms voice).freeze

 params do
  required(:starts_at).filled(:string)
  optional(:rsvp).filled(:string, included_in?: RSVP)
  required(:type).filled(:string, included_in?: TYPE)
  rule(:starts_at) do
    failure(:invalid) unless values[:starts_at] =~ FormatConstraints::ISO8601_DATETIME
  end
end

API::V2::AppointmentContract.new.({type: "sms", starts_at: "2019-09-01T10:00:00Z+03:00"})

I get the following error:
Traceback (most recent call last):
        4: from (irb):26
        3: from app/contracts/api/v2/appointment_contract.rb:1:in `<main>'
        2: from app/contracts/api/v2/appointment_contract.rb:5:in `<class:AppointmentContract>'
        1: from app/contracts/api/v2/appointment_contract.rb:6:in `block in <class:AppointmentContract>'
NoMethodError (undefined method `visit_nominal' for #<Dry::Schema::PredicateInferrer::Compiler:0x00007fbf616abbc0>)
Did you mean?  visit_enum

It's because of dry-types 0.15. Please use 0.14 for now, we'll release a new alpha/beta of dry-schema soon. Alternatively, you can work with master branches of dry-{validation,schema}, they are already compatible with 0.15.

@flash-gordon thanks! that works

@danielnc I'll be releasing 1.0.0.beta1 later this week

somewhat unrelated but I've been exploring 1.0
Where can I find examples for nested arrays?
On the specs I could only find for hash values
Feel free to close this issue if this is not the place to ask this kind of question

Where can I find examples for nested arrays?

For questions like this you can join either our chat (we're testing out Zulip) http://dry-rb.zulipchat.com or post a message on https://discourse.dry-rb.org


MixFormatDiff folding

Could you tell me if diff should fold unchanged text or it shouldn't? Thanks!

Well, If after MixFormatDiff command I issue diffthis I get what I need :)

Is this my setup lacks this last command call within MixFormatDiff or this is intended behaviour -- I and other users of MixFormatDiff should manually diffthis if needed?

Ok, this could be solved by the User autocmd:
autocmd User MixFormatDiff diffthis | wincmd p | diffthis

Just a question though, is it intended behavior and we should use autocmd?

Hmm, by that time :diffthis was done for both buffers already (you can check with :set diff?). The behaviour you see is caused by this commit.
It's a bad commit. I can't remember why I did this and the commit doesn't say so. :\
I revert it for now!


Final Check of the items in the checklists

@AdeptiC, @Jakobloula, @Zeruxe, @Akbar-163, and @zakariyams
You completed the required items for the 'Laboratory Work' and received 4.5 credits.
Well done! 🎉


The api-version parameter to start a job should be passed back in the Operation-Location header

While working on the authoring client, we found that long-running operations (LROs) were 404'ing during status updates because the api-version query parameter was not returned. Talking with @JeffreyRichter, the Operation-Location for LROs (and nextLink for paged results) should be opaque, so they need to include the api-version parameter - the same one passed during the request to start a job or get paged results.
@ChongTang could you also check if CLU might have this problem?
I'll also be opening bugs that generated code should add it if missing based on the client options, but ideally services do this consistently so any caller doesn't have to work around the issue.
/cc @annatisch @AhmedLeithy @ZulaMostafa @mshaban-msft

While it doesn't necessarily help in this situation, @johanste also says that the service shouldn't 404, but 400 (bad parameter) if api-version is required and not specified.

I'll be looking into it for QuestionAnswering, planning a rollout of the fix by 12/3.


NoHttpResponseException

I am using secor secor-0.22-SNAPSHOT.jar
every 20-30 minutes i am getting the following error and process stop
during the 30 minutes that secor is working , its working very good
2016-10-06 21:33:15,204 [Thread-8] (com.pinterest.secor.main.ConsumerMain$1:65) ERROR Thread Thread[Thread-8,5,main] failed
java.lang.RuntimeException: Failed to apply upload policy
at com.pinterest.secor.consumer.Consumer.checkUploadPolicy(Consumer.java:116)
at com.pinterest.secor.consumer.Consumer.run(Consumer.java:106)
Caused by: java.util.concurrent.ExecutionException: java.lang.RuntimeException: org.apache.http.NoHttpResponseException: ia-staging-parquet.s3.amazonaws.com:443 failed to respond
at java.util.concurrent.FutureTask.report(FutureTask.java:122)
at java.util.concurrent.FutureTask.get(FutureTask.java:188)
at com.pinterest.secor.uploader.FutureHandle.get(FutureHandle.java:34)
at com.pinterest.secor.uploader.Uploader.uploadFiles(Uploader.java:108)
at com.pinterest.secor.uploader.Uploader.checkTopicPartition(Uploader.java:220)
at com.pinterest.secor.uploader.Uploader.applyPolicy(Uploader.java:252)
at com.pinterest.secor.consumer.Consumer.checkUploadPolicy(Consumer.java:114)
... 1 more
Caused by: java.lang.RuntimeException: org.apache.http.NoHttpResponseException: ia-staging-parquet.s3.amazonaws.com:443 failed to respond
at com.pinterest.secor.uploader.HadoopS3UploadManager$1.run(HadoopS3UploadManager.java:68)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
Caused by: org.apache.http.NoHttpResponseException: ia-staging-parquet.s3.amazonaws.com:443 failed to respond
at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:143)
at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:57)
at org.apache.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:261)
at org.apache.http.impl.AbstractHttpClientConnection.receiveResponseHeader(AbstractHttpClientConnection.java:283)
at org.apache.http.impl.conn.DefaultClientConnection.receiveResponseHeader(DefaultClientConnection.java:251)
at org.apache.http.impl.conn.AbstractClientConnAdapter.receiveResponseHeader(AbstractClientConnAdapter.java:223)
at org.apache.http.protocol.HttpRequestExecutor.doReceiveResponse(HttpRequestExecutor.java:272)
at org.apache.http.protocol.HttpRequestExecutor.execute(HttpRequestExecutor.java:124)

I've seen this a few times when there were some network issues uploading to S3. The nice thing is that secor will retry the upload and only commit the latest offsets to ZooKeeper if the upload succeeds.

Is there a solution ? like config number of retry upload ? can Secor recover from this ?
What I am currently doing is run secor again till next failure ,  I added watchDog that will start the process in case its down but its not a valid option in production

Will the S3UploadManager works better for you?
On Fri, Oct 7, 2016 at 6:01 AM, Dori Waldman<EMAIL_ADDRESS>wrote:

Is there a solution ? like config number of retry upload ? can Secor
recover from this ?
What I am currently doing is run secor again till next failure , I added
watchDog that will start the process in case its down but its not a valid
option in production
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/261#issuecomment-252245873,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHCX_7lXm5iz5J2RYJkWGLuPeDD2TLqhks5qxkKUgaJpZM4KQ8F8
.

Thanks looks like switch from default to S3UploadManager fixed it , its now running for 14 hours , with the default it crashed every hour
Thanks


Qualify size_of in drain.rs

"size_of" either needs to be imported or qualified. Chose to qualify to match convention in file.

Thanks! prelude::size_of stable on 1.80.0


Chat view padding issue removed

Details
Implemented Solution is:
Go to ReportActionCompose.js File. Go to Parent View then there are multiple containers i.e. attachments, send button, and input text field. There was a problem with the input text field view styles. There wad Unnecessary padding of the input text field which was causing the extra space at the top and was not setting the input text view to the center of the parent view So we removed that and input text is at the center of the parent view and the top spacing has been removed using top, bottom padding value to 0 and I have tested this.
I am waiting to add me to the git link SO I can push the code to my branch and your team can review the code.
Fixed Issues
Remove Top Padding from chatView.
Tests
Tested on Android OnePlus 3 Mobile.
Tested on iOS iPhone X emulator.
Tested On

[x] iOS
[x] Android

iOS

Android

Hi @muhammadusama95 - just a few early comments on this PR:

Please include the link to the issue you've been assigned under the Fixed Issue header
Please be sure to test any and all changes on all platforms, including screenshots (web, desktop, mobile web, iOS, Android). Right now, I only see iOS and Android
Looking at the screenshot you provided on Android, the text in the input field is vertically misaligned

Hi, @roryabraham Thanks for your review of my code.
I just made a new commit, Added Comments on code where I changed for the fixing of the top padding issue. So you can easily
identify my code.
Below are my answers to all your questions:
1: https://github.com/Expensify/Expensify.cash/issues/1002
2: Yeah As per my discussion with Michael, This is a cross-platform app So I have tested it on both iOS and Android. Not included web, desktop, or mobile web version.
3: By mistakenly added an old android screenshot Please see attached updated android screenshot too.

4: Now I added comments in the new commit So you can quickly identify my changes.
5: There is no new lint error or warnings in my branch as I just changed the values of padding.

Hi @roryabraham,
Please check the last 3 commits So you can check the indentation Set, Comments removed, and padding issue fixed.
Now I have tested it using login through Expensify app and check below screenshots of both iOS and Android respectively.

About testing it on All Platforms @michaelhaxhiu had chat with me on android and iOS only. Check Below Screenshot.

Hi @muhammadusama95. Regardless of what others may have told you, all PRs must be tested on all platforms. So that means that your code changes must be tested on:

Web (from a laptop)
Desktop (from a mac computer - let me know if you don't have a mac)
Web (from a mobile device or simulator)
iOS
Android

Hi @roryabraham,
Please find All Platform tested screenshots.
1: Android

2:iOS

3:Mobile Web

4:Web

5:MAC-Desktop

Hi @roryabraham,
Please find All Platform tested screenshots.
1: Android

2:iOS

3:Mobile Web

4:Web

5:MAC-Desktop

Looks like you've got a merge conflict

Looks like you've got a merge conflict

Let's give @roryabraham a chance to review, as he's been doing 🔧

Let's give @roryabraham a chance to review, as he's been doing 🔧

@muhammadusama95 There are still lots of unresolved comments that would need to be addressed before this could be merged. Not to mention there's a merge conflict, so it's not even possible to merge presently.

@muhammadusama95 There are still lots of unresolved comments that would need to be addressed before this could be merged. Not to mention there's a merge conflict, so it's not even possible to merge presently.

@michaelhaxhiu @roryabraham Can you please assign this issue to<EMAIL_ADDRESS>I will create a new branch and take a pull request instead of resolving these comments by doing this it's hardly 20-25 minutes work.

@michaelhaxhiu @roryabraham Can you please assign this issue to<EMAIL_ADDRESS>I will create a new branch and take a pull request instead of resolving these comments by doing this it's hardly 20-25 minutes work.

On the new branch, you will get only the required changes. Nothing else.

On the new branch, you will get only the required changes. Nothing else.

@muhammadusama95 We have decided to move forward with another contributor. Thanks for your assistance thus far, but no further action is required from your side.
I'm closing this PR and going to pay you in Upwork shortly.

@muhammadusama95 We have decided to move forward with another contributor. Thanks for your assistance thus far, but no further action is required from your side.
I'm closing this PR and going to pay you in Upwork shortly.


[FLINK-14721][hive]HiveTableSource implement LimitableTableSource interface

What is the purpose of the change
HiveTableSource implement LimitableTableSource interface
Brief change log
HiveTableSource implement LimitableTableSource interface and add tests for it.
Verifying this change
This change added tests and can be verified as follows:

Added HiveTableSourceTest#testLimitPushDown

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): ( no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no )
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes )
If yes, how is the feature documented? (not applicable)

cc @xuefuz @bowenli86 @lirui-apache to have a review.

@zjuwangg could you rebase to master?

LGTM, merging


TranslationSystem API & reload i18n module assets

Contains
Adds i18n support for modules
How to test
Create a module TestAndTryout (or any other name, replace the translation key if so) with the following system:
@RegisterSystem
public class GreeterSystem extends BaseComponentSystem {

    private static final Logger logger = LoggerFactory.getLogger(GreeterSystem.class);

    @In
    private TranslationSystem translationSystem;

    @ReceiveEvent
    public void onPlayerSpawn(OnPlayerSpawnedEvent event, EntityRef player) {
        logger.info(translationSystem.translate("${TestAndTryout:greeter#greet}"));
    }
}

add to the module assets/i18n (create folder):
greeter.lang
{
   "greet": "greet"
}

greeter_en.lang
{
   "greet": "Hello new player."
}

greeter_de.lang
{
   "greet": "Hallo neuer Spieler."
}

Without the PR the system will not load (system is not api)
With the PR, start a game with the module and english language setting, output should be "Hello new player", then change language to german and start another game, output should now be "Hallo neuer Spieler".

Outstanding before merging
The refresh system feels a bit "hacky" but i found no better place in the I18nSubsystem to register for the game to be started. I would be fine to merge it the way it is though.

Hooray Jenkins reported success with all tests good!

I added an todo link to #2433 which is the closes issue so far.
Documentation update is included in the new docs, currently at https://oniatus.github.io/workflow/translation.html
We can open an issue with a follow-up tutorial module for i18n, probably bite-size and/or a short comment on #2433 but the pr itself is okay from my side.

Hooray Jenkins reported success with all tests good!

Tested it out in a module - works as advertised! Very much looking forward to putting that to more use one day :-)
Added a quick entry to the Core module's build.gradle (and its siblings) to generate the new directory when needed


[REQUEST] Manual IP Configuration

Could not properly get my Nanoleaf Shapes to connect to the application. Would appreciate a manual connection configuration option where I have the availability to enter the proper IP and port of my Shapes.

I have this planned out as a feature for the new version of this program, however, this version program isn't compatible with Shapes anyway so typing in the IP and port manually wouldn't do much good.


ERROR in   Error: Child compilation failed:   Module build failed: Error: The path for file doesn't contain relativeTo param

I'm new into webpack and trying move gulp to webpack.  facing issue in "ngtemplate-loader". Below is my webpack config.
const path = require('path');
const webpack = require("webpack");
const optimize = webpack.optimize;
const htmlWebpackPlugin = require('html-webpack-plugin');
const ExtractTextPlugin = require('extract-text-webpack-plugin');
const CopyWebpackPlugin = require('copy-webpack-plugin');
const config = require('./webpack/config');
var webpackConfig = {
entry: {
    vendor: vendorResources,
    app: ["./src/app/index.module.ts"]
},
output: {
    path: path.resolve(__dirname, "dist"),
    filename: "scripts/[name].js",
},
devtool: "source-map",
module: {
    rules: [{
            test: /\.html$/,
            use: [
                { loader: 'ngtemplate-loader?relativeTo=' + (path.resolve(__dirname, './src/app')) },
                { loader: 'html-loader' }
            ]
        },
        {
            test: /\.ts?$/,
            use: [{
                loader: 'awesome-typescript-loader'
            }, {
                loader: 'tslint-loader'
            }],
            exclude: /node_modules/
        }
    ],
},
resolve: {
    extensions: [".ts", ".tsx", ".js", ".json", ".jsx", ".css", ".scss", ".sass"]
},
externals: {},
plugins: [
    new ExtractTextPlugin({
        filename: (getPath) => {
            return getPath('styles/[name].css').replace('js', 'css');
        },
        allChunks: false
    }),

]

}
module.exports = webpackConfig;

Hi @karthikdeva , have you solved this issue?

Not sure if this is the original issue, but for me my test for /\.html/ was also picking my index.html file from my html-webpack-plugin. Putting an exclude: /index.html/ did it for me. You may have to adjust the exclude if you have other files names index.html, but the basic gist is:
module.exports = {
  ...
  module: {
    rules: [
      {
        exclude: /index\.html/,
        test: /\.html$/,
        use: [
          {
            loader: 'ngtemplate-loader',
            options: {
              relativeTo: './src'
            }
          },
          { loader: 'html-loader' }
        ]
      }
    ]
  },
  ...
}

@nickbreaton It worked for me, thanks!

Is this maybe only windows problem?

No, I have linux and experiencing same issue.


simplify DeepCopy implementation

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
ec75abe19c5458fb948f0947445058a05fa79774
link
true
/test gencheck

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-sigs/prow repository. I understand the commands that are listed here.

@wulianglongrd: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
ec75abe19c5458fb948f0947445058a05fa79774
link
true
/test gencheck

lint_istio
ec75abe19c5458fb948f0947445058a05fa79774
link
true
/test lint

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-sigs/prow repository. I understand the commands that are listed here.

@wulianglongrd: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
ec75abe19c5458fb948f0947445058a05fa79774
link
true
/test gencheck

lint_istio
ec75abe19c5458fb948f0947445058a05fa79774
link
true
/test lint

integ-basic-arm64_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-basic-arm64

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-sigs/prow repository. I understand the commands that are listed here.

@wulianglongrd: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
ec75abe19c5458fb948f0947445058a05fa79774
link
true
/test gencheck

integ-basic-arm64_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-basic-arm64

integ-distroless_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-distroless

integ-cni_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-cni

integ-pilot-istiodremote-mc_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-pilot-istiodremote-mc

integ-pilot-multicluster_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-pilot-multicluster

integ-ds_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-ds

integ-security_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-security

integ-telemetry_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-telemetry

release-notes_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test release-notes

lint_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test lint

macbuildcheck_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test macbuildcheck

integ-security-multicluster_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test integ-security-multicluster

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-sigs/prow repository. I understand the commands that are listed here.

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
unknown
/test integ-cni

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-sigs/prow repository. I understand the commands that are listed here.

/retest-required

@wulianglongrd: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
unknown
/test integ-cni

unit-tests-arm64_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
unknown
/test unit-tests-arm64

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-sigs/prow repository. I understand the commands that are listed here.

/test integ-cni

/test unit-tests-arm64

@wulianglongrd: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
unknown
/test integ-cni

unit-tests-arm64_istio
47862667332a6aff0875a907f7a791c4ff3b59e1
link
true
/test unit-tests-arm64

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-sigs/prow repository. I understand the commands that are listed here.

/retest


Project deletion failed if not empty

Describe the bug
Terraform failed to delete a project if this one is not empty.
As image are replicated in a projects by a replication, the deletion routine should also clean the project before deletion
An other solution can be to create a resource that liunk to the sub-object and potential replication.

There is a documented feature for this behaviour called force_delete.
But there was no error-handling in the code to catch a terraform destroy when force_delete wasn't set.
I have written a PR that should solve this. https://github.com/goharbor/terraform-provider-harbor/pull/357


Add query string to type definition (gatsby-config.js) in order to handle i18n

@alexandrebodin there we go, I applied your idea that was indeed the best way to handle this problem.
I've updated README accordingly.
I've also added queryLimit param to this new behavior. It can now be overriden individually for each type.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Hugues BUREAU seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Is the CLA signature ok? I've done but does not seem to have worked, still asking for it on this page!

@remidej you were right, I've been missing pluralization if no endpoint provided, this is fixed now. (So I've not updated the example in the README, will work just fine now).
README typos fixed too

Hi guys !
Thank you for your work :D
Do you have an idea for when it will be usable and published in npm ? :)
Thank you !

Thanks again @8byr0
@juliensl we'll publish this as part of a new alpha release on NPM early next week. We're just waiting for a couple more changes we'd like to add to that release

So great ! Thank you @remidej and thank you too @8byr0  :smiley:


Inline CSS variables are not properly applied

Have a look at the following example:
@Composable
fun Demo() {
    Style(DemoStyles)
    Div({
        classes(DemoStyles.container)
        style {
            DemoStyles.color(Color.red)
        }
    }) {
        Text("Hello World")
    }
}

object DemoStyles : StyleSheet() {
    val color by variable<CSSColorValue>()

    val container by style {
        backgroundColor(color.value())
    }
}

This produces the following markup:

Hello World

The expected output is:

Hello World

As you can see, the -- is missing, which turns the variable into a standard CSS property. When I move the DemoStyles.color(Color.red) down to the stylesheet, everything works as expected, so the issue only occurs in inline styles.
Workaround:
        property("--${DemoStyles.color.name}", Color.red)

Just throwing in my own 2 cents, as a +1 to this bug.
I finally just started looking into supporting variables in my own project, and I thought it was my fault at first that inline variables weren't working.
Here's an even simpler example that I believe should work:
Div(attrs = {
   style {
      variable("testvar", 10.px)
   }
})

This should generate the element:

but it drops the variable in this case, generating just:

See also: https://ishadeed.com/article/css-variables-inline-styles/

Same here
Expected: After applying a variable color, when getting the variable it should return that color (Background should be blue)
Actual: The variable returns nothing or the default value even if you applied another color before (Background is red)
`
fun main() {
renderComposableInBody {
SomeComposable()
}
}
@Composable
fun SomeComposable() {
Style(MyStyle)
Div({
    classes(MyStyle.myDiv)
}) {
    Text("SomeComposable")
}

}
private object MyVariables {
val contentBackground by variable()
}
private object MyStyle : StyleSheet() {
val myDiv by style {
MyVariables.contentBackground.value(Color.blue)
backgroundColor(MyVariables.contentBackground.value(Color.red))
}
}
`

I ended up reimplementing keyframe styling logic in my own project. I can confirm that the first comment's workaround works, just use property("--$name", ...) instead of the documented method for specifying variable values.
If your project isn't that far into using Compose HTML, you might consider taking a look at my project Kobweb, since it seems like Compose HTML isn't receiving fixes very quickly, and by being a layer on top of theirs I can fix things before they do: https://github.com/varabyte/kobweb#css-variables

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


reduce font-size list items to @font-size-small

Yes I saw that just after I replied but was just about falling asleep by
then!
On Sun, 3 Nov 2019, 05:31 Virgil<EMAIL_ADDRESS>wrote:

@virgilwashere commented on this pull request.
In theme/stylesheets/website/summary.less
https://github.com/mautic/documentation/pull/350#discussion_r341833750:

@@ -32,6 +32,7 @@

         li {
             list-style: none;

           font-size: @s-font-size;

@s-font-size:                         1.2rem;@l-font-size:                         1.6rem;@default-font-size:                   @l-font-size;
At the bottom of the same file is where my tracing got me, as the
inheritance goes all the way to @default-font-size.
However, @font-size-small all nice and prominent at the top of the file,
does seem more user-friendly than @s-font-size
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mautic/documentation/pull/350?email_source=notifications&email_token=AAWLPIL7TRJ7QL2VXLDDZ5DQRZAZNA5CNFSM4JIB4TO2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCKDGNRI#discussion_r341833750,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWLPIITDEQW4352TPZY7C3QRZAZNANCNFSM4JIB4TOQ
.


Your detection2 is missing the code required for building

Your detection2 lacks the necessary code for building, which can cause errors such as/home/dgp/code/PLU/detection2/layers/csrc/vision. cpp: 4:10: fatal error: ROIAlign/ROIAlign. h: No such file or directory
4 | # include "ROIAlign/ROIAlign. h"
|^~~~~~~~~~~~~~~~~~~~~
Compilation terminated
Error: command '/usr/bin/gcc' failed with exit code 1

Detectron2 was only successfully installed after I added the necessary code！

Thanks for your issue, I will check it.


Deploying using the now button does not work

I just tried it out. I created a new token and pasted it into the field, htting deploy afterwards. The webpage tells me Deploy failed, please check the repo and try again.. It worked for the test repository https://deploy.now.sh/?repo=https://github.com/zpnk/hello-world that was linked on the same page with the same token I tried out earlier.
Deploying using Docker didn't work either.

Hey @SirWindfield
Be sure to specify all of these environment variables as part of the deploy:
    "env": {
      "TWITTER_CONSUMER_KEY": "@twitter-consumer-key",
      "TWITTER_CONSUMER_SECRET": "@twitter-consumer-secret",
      "TWITTER_ACCESS_TOKEN_KEY": "@twitter-access-token-key",
      "TWITTER_ACCESS_TOKEN_SECRET": "@twitter-access-token-secret",
      "LOGS_SECRET_PREFIX": "@logs_secret_prefix"
    }

I just updated the button such that you are now required to enter them during the deployment.
Also be sure to check the [ ] Build using Docker button.
I am going to close this for now. Please let me know if you run into other issues. 😄


Fetch token balances using Blockscout

Description
Before this PR when fetching balances we were querying the balance directly from the token contracts.
This has the advantage that it's querying directly from Forno or other nodes that are more likely to be up to date than blockscout, which sometimes has a delay to update balances.
However, the disadvantage was that we were making one request per each token. If there are not a lot of tokens this is fine, but there are 42 tokens already on https://github.com/Ubeswap/default-token-list/blob/master/ubeswap.token-list.json and this number will continue to grow. Making so many requests each time we want to fetch balances looks like too much.
Another possible solution longer-term is indexing and updating the balances of our users ourselves using the indexer. This would give us control over the latency for block updates while not having to make so many requests at the same time.
Other changes

Created an action which re-fetches the token balances and called it in the places where the old ones used to be called.
Created an error action to be used in the UI when there are errors in future PRs.

Tested
Manually and with unit tests.
How others should test
Nothing to test directly here, just make sure the balances show up and are up to date after making a transfer.
Related issues

Part of #1126

Backwards compatibility
N/A

Could we avoid this change by using batch requests?
If we were to go with fetching the balance from blockscout, what do you think of abstracting it through blockchain-api?

I agree with routing through blockchain-api, I'll make that change.
I don't see how to batch requests using contractKit unfortunately and since there could be a large number of tokens getting the balance for all of them client-side doesn't seem very data-efficient :/

@jeanregisser Here's the PR on blockchain-api: https://github.com/valora-inc/blockchain-api/pull/42 I updated this one to use it as well, it requires it to be deployed to work


Is there any plan to support knex 0.15?

Now bookshelf's package marks that it need knex version >=0.6.10 <0.15.0.
https://github.com/bookshelf/bookshelf/blob/2afb78eeeae1c1c33453161215400bd1ed2bf834/package.json#L49-L51
Which issue block the update?

Yes, Knex 0.15+ support is planned for the 1.0 release of Bookshelf. That will happen after the next version (0.14) is released. See here for more info.

Closing due to lack of response.


[CPP-434] Detect signed overflow checks

I think this overlaps with cpp/bad-addition-overflow-check when the check is done on two small signed integers. Should it be a modification to that query to look for signed types as well as small types?

Ran the query against a larger (66) suite of projects, and there are too many false positives for it to be usable.

I think this overlaps with cpp/bad-addition-overflow-check when the check is done on two small signed integers. Should it be a modification to that query to look for signed types as well as small types?

Here is what @jbj wrote in the Jira ticket:
cpp/bad-addition-overflow-check also flags expressions that look like x + y < x, but it's looking for a very different problem. It applies to both signed and unsigned types but only to types smaller than int.

I think @rdmarsh2 is right. These queries will have to be merged before we can put them in production, but during initial development it's probably easier to keep them separate.
@zlaski-semmle Are there false positives because the query isn't good enough yet or because it turns out there are cases where it's defined behaviour to check x + y < x for signed addition and positive y?

@zlaski-semmle Are there false positives because the query isn't good enough yet or because it turns out there are cases where it's defined behaviour to check x + y < x for signed addition and positive y?
The former.  But I've improved the query since.

Looking at the results on 78 projects they look pretty good to me now.  This was an interesting case:
if ((int)(ioff + c.length) < (int)ioff) return 0;

because the addition is done on unsigned integers (so is OK), before converting to signed for the comparison (so it's the conversion that may produce undefined behaviour).
Part of me also wants a sister query that flags testing for signed overflow without a comment explaining what's going on.  :)

because the addition is done on unsigned integers (so is OK), before converting to signed for the comparison (so it's the conversion that may produce undefined behaviour).

@geoffw0, I think conversion from unsigned to signed is well defined even when the value doesn't fit. From https://en.cppreference.com/w/cpp/language/implicit_conversion:

If the destination type is signed, the value does not change if the source integer can be represented in the destination type. Otherwise the result is [implementation-defined (until C++20)] [the unique value of the destination type equal to the source value modulo 2^n
where n is the number of bits used to represent the destination type. (since C++20)]. (Note that this is different from signed integer arithmetic overflow, which is undefined).

So it's implementation-defined what happens, and I'm guessing that all implementations define that they translate the bit representation into two's complement.

I cooked up another test case:
bool multipleCasts2(char x) {
    // clang 9.0.0 -O2: not deleted
    // gcc 9.2 -O2: not deleted
    // msvc 19.22 /O2: not deleted
    return (int)(unsigned short)(x + '1') < (int)(unsigned short)x; // GOOD [FALSE POSITIVE]
}

This one is not deleted by any compiler -- I'm guessing because there is only one top-level expression on the left side, and so we have a plain comparison instead of an overflow check.  Since our AST implementation strips away casts, QL only sees x + '1' < x, hence the alert.  But given that this example is rather contrived, I wouldn't worry about it.

I've just pushed out another rewrite of Qhelp.  Please review content, but also FORMATTING.  I'm especially concerned about the table of constructs in the Recommendation section.  The MD result renders fine in my Chrome browser with a mark-down plug-in.  I'm attaching my MD generation script as well.
qhelp_to_md.txt

Here are the errors from QHelp-Preview: https://gist.github.com/jbj/3cc7d8909268195858b5d29f76c05a3d

What's next? You're still investigating how to suppress the unhelpful results we saw in the query console run, right?

Yes, once I get done with the documentation formatting, I'll tackle the exclusion of macro invocation from alerts.

Latest results at https://lgtm.com/query/6923897885970565767/.  We did manage to squelch alerts for macro invocations.

So my check-in died in Qhelp preview again, but I cannot see what's wrong since I don't have access to Jenkins.

I added a commit to insert <p> tags in the qhelp.

The qhelp LGTM.
I added
  ... and
  forall(Compilation c |
    c.getAFileCompiled() = ro.getFile()
  |
    not c.getAnArgument() = "-fwrapv" and
    not c.getAnArgument() = "-fno-strict-overflow"
  )

to suppress the results on Linux and CoreCLR. Here are the remaining results: https://lgtm.com/query/4587217464269309481/. What do you say to incorporating that forall into the query? I haven't tested for performance.
I'm still not sure what to think about the x + 1 >= 1 tests that also show up among the results. They don't seem to be dangerous but are just convoluted ways of writing x >= 0, and so I'm not sure they belong in the results of a security query. What do you think? They could be suppressed by requiring that expr2 is not a constant.

I added a commit to insert <p> tags in the qhelp.

Thank you for doing this.  For some reason, the Qhelp rendered correctly when using my local QHelp-to-MD script.

I'm currently running timing tests (-fwrapv / -fno-strict-overflow vs. without).  I'm on the fence as to whether alerts should be produced even in the presence of these command-line parameters.  My only concern, really, is that the build system may get inadvertently changed at some point.  Aside from that, though, I'm OK for incorporating the conjunct you provided.

So I've run the tests, and judging by wall-clock, cold-cache running time, I don't actually see any performance degradation as a result of adding the -fwrapv / -fno-strict-overflow checks.  Of course, the margin of error is quite large here, but I think it's safe to say that performance is not something we should worry about.
@jbj I'm thinking that the forall in your conjunct should perhaps be replaced by exists?  I'll try an LGTM run with that change.

I'm still not sure what to think about the x + 1 >= 1 tests that also show up among the results.

Actually, I don't see such results.  Where did you see them?  The closest I found is i-binwidth*k+1 >= 1 in igraph/igraph - perhaps that's what you meant?  But I'd guess that, by writing x + 1 >= 1, the developer is testing for overflow, which the much simpler x >= 0 would not accomplish.  In fact, I believe the correct recommendation here would be x < MAX_INT.

I'm thinking that the forall in your conjunct should perhaps be replaced by exists?

Good catch. I did indeed have this the wrong way around.

I think this is good to merge now. Thank you for your patience with my many rounds of reviewing. We can decide about the x + 1 >= 1 results when we see some more of them in the wild.


Deployment documentation

Task

[x] Document the Elastic Beanstalk deployment for the report service

Outcome
We will have documentation on the Elastic Beanstalk deployment.

Initial version of the README.md has been created. I suspect updates will be required after the rebuild, but we can tackle those as needed.


Use cli-testlab for testing CLI

Here is a bit of mad science that I was busy with lately: a better, more seamless and way less verbose way to test CLI (still using Jake, but in a more embedded way).
This is to serve as a PoC of sorts. Should this be merged, I'll migrate all remaining cli tests to cli-testlab and remove knex internal CLI testing microframework

Would be nice to include also tape / jake tests to be run with coverage to have real coverage numbers

@elhigu Will look into tape one, but with Jake don't think it's that simple, as it is executed in a separate process still.


Correctly format API error on image pull

Recent changes caused some API errors to not be properly formatted. In the case of pull, the job output used to always be formatted in JSON, but since ditching the job framework for this operation everything except errors were being formatted correctly.
Now, where we had this on docker 1.6.0:
$ docker pull aoeuaoeu
Pulling repository aoeuaoeu
FATA[0003] Error: image library/aoeuaoeu:latest not found

And then had this on master (1.7.0-dev)

We now have this again:
$ docker pull aoeuaeu
Pulling repository aoeuaeu
FATA[0002] Error: image library/aoeuaeu:latest not found

ping @jfrazelle

I'm curious, what does the old vs new format look like?

@duglin see https://github.com/docker/docker/pull/12640#issue-70156595 for what the client displays on pull error

close closes #12640
can you run
curl -sSL https://github.com/jfrazelle/docker/commit/d4e00ee05455a561eb8e8425a63b8dded359f32f.patch | git am

and it will apply my patch to your branch

thanks @jfrazelle

LGTM as long as janky likes :)

which would be my fault if otherwise bc that was my test

it failed on DockerSuite.TestPushMultipleTags which is a known bad one so no worries

@jfrazelle right, is it TestPushInterrupt (fix on https://github.com/docker/docker/pull/12621) that it races with?

yeah
On Wed, Apr 22, 2015 at 1:31 PM, Josh Hawn<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle right, is it TestPushInterrupt
(fix on #12621 https://github.com/docker/docker/pull/12621) that it
races with?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/12649#issuecomment-95326424.

@jlhawn found a diff better fix so wait to merge :)

@jfrazelle @duglin updated!

Also, here's where the issue was introduced: https://github.com/docker/docker/commit/6e38a53f96403b4cbd38e49e6b294128ed054a20?diff=unified#diff-44a527ea72ea9572e1cf71c7f1e2a0f0L687

LGTM

ping @duglin

LGTM


RAID interactive

Please bring back the RAID interactive activity, good interactive and gets students thinking about the parity bit tasks in a creative way.

Thanks for the suggestion. We are busily preparing for the release of version 3 of the Field Guide so cannot consider your request just yet, but will certainly do so after

more information at https://oneteachersview.blogspot.com/2013/08/coding-homework.html

Some observations and thoughts:

This interactive was created just over 6 years ago for http://computerscienceclub.org/ (no longer exists) and "it will end up in the Field Guide".

The earliest official archive of the field guide (1.9.9) is from just under 5 years ago, and doesn't appear to have it.
So either it didn't end up on the Field Guide or only did for a short period.

The interactive appears to focus on singular parity bits, i.e. those used for error detection. But with the extra information of which drive failed, it is used for error correction.

In the Error Control chapter, the concept of parity bits for error detection is almost entirely skipped in favour of 2D parity.
As such, people could erroneously interpret the ability of 1D parity to be used in error correction under all situations.

Compared to the existing Parity interactive, this one focuses much more on a real-world scenario.
It is also faster; with only one dimension of parity to worry about (and not needing to define every parity bit), users can progress through multiple repetitions of the exercise in the same time it would take just one iteration of the Parity interactive.
The interactive itself appears very simple and can probably be replicated with little effort.


Support mobile

Thank you for your work! Is it possible to make this plugin support Android version please?

Hi there! I don't think that would be possible, since the plugin uses the wolfram engine and executes commands, things that you can't do on phones as far as I know...


How to get pipeline task status for a specific task?

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Question, Bug, or Feature?
Question
Enter Task Name: Status info of a specific pipeline task
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure DevOps Pipelines

Agent - Hosted or Private:
Hosted agent running multiple tasks in a job

Issue Description
My purpose is to get the status of a specific pipeline task in a job.
I've tried to send an API call referencing the method at the https://docs.microsoft.com/en-us/rest/api/azure/devops/release/releases/get-task-log?view=azure-devops-rest-6.0&tabs=HTTP  but it only returned the running log of the task.
Is there exist any method to get the status information of a pipeline task?
Can we achieve to get status for one task?
Any suggestions and directives is appreciated.
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

Any update on the solution method of the task?


LibArtifact Data-1.0.lua:174

Hi, i've this problem after 5 minutes of run script, is it possible to fix it?
Message: NerdPack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:174: attempt to index local 'info' (a number value)
Time: 07/11/17 01:35:28
Count: 1
Stack: ...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:174: in function <...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:167>
...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:242: in function <...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:233>
...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:258: in function <...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:254>
...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:321: in function `?'
...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:68: in function <...ack\Libs\LibArtifactData-1.0\LibArtifactData-1.0.lua:68>
Locals: artifactID = nil
traits =  {
}
powers =  {
1 = 40
2 = 41
3 = 42
4 = 43
5 = 44
6 = 47
7 = 49
8 = 50
9 = 51
10 = 52
11 = 53
12 = 54
13 = 350
14 = 351
15 = 352
16 = 353
17 = 1118
18 = 1275
}
(for index) = 1
(for limit) = 18
(for step) = 1
i = 1
traitID = 40
info = 205273
(*temporary) = 40
(*temporary) = 205273
(*temporary) = 105280
(*temporary) = 1
(*temporary) = 1
(*temporary) = 0
(*temporary) = 0.72323298454285
(*temporary) = "attempt to index local 'info' (a number value)"
GetPowers =  defined =[C]:-1
GetPowerInfo =  defined =[C]:-1
GetSpellInfo =  defined =[C]:-1
artifacts =  {
knowledgeMultiplier = 1
knowledgeLevel = 0
}

Related: https://github.com/MrTheSoulz/NerdPack/issues/218

Related: https://github.com/MrTheSoulz/NerdPack/issues/218

Resolved...
i've updated the libartifact data to 1.1.1 version from wowcurse ;)
https://mods.curse.com/addons/wow/libartifactdata-1-0#t1:other-downloads
ty ^:D

I've downgraded the one included with NeP from 1.2 to 1.1.1
thanks for the tip!


Putting < in the world name causes a crash

Looks like OTG allows characters for the world name which cannot be used in windows folder names which causes CTD on world creation.

Normally, Minecraft will substitute invalid symbols with an underscore in the file name. OTG does not do this, causing the crash.

For v9.0_r8, only "^[a-zA-Z0-9 _-]+$" are allowed, continue button is unavailable until name complies.


Update Kademlia API. Fix DiscoveryBehaviour delays.

This is a draft that adapts the Kademlia code to upcoming changes from https://github.com/libp2p/rust-libp2p/pull/1174 and https://github.com/libp2p/rust-libp2p/pull/1189. Also contains a fix for the artificial discovery delays, which will be backported to the current master in a separate PR, since this PR only becomes relevant with the next libp2p release at the earliest.

Superseded by https://github.com/paritytech/substrate/pull/3076.


Change beautify.ignore scope to resource

This is a simple change to enable folder configurations in multi-root workspaces to set their own ignored files. It's pretty air-weight, especially since you're already calling getConfiguration on the workspace. If you incorporate that it would save me a lot of headaches in onboarding new teammembers and such.

Ok, cleanup changes made! Thanks for you patience!


CI tests for starknet-forge template

It should include running:

forge test (using forge github action)
scarb fmt --check

Blocked by https://github.com/foundry-rs/starknet-foundry/issues/396

No longer relevant since we move the template to our repo in #515


Issue with packaging in latest version

Description
I just installed the following packages in a clean conda environment with python3.8 and I am running into the following error message. It's pretty easy to duplicate.
Packages
tiktoken==0.4.0
zep-python==1.1.0
packaging==23.1

Error Trace
zep-seeder  | Traceback (most recent call last):
zep-seeder  |   File "/app/setup.py", line 9, in <module>
zep-seeder  |     client = ZepClient(base_url=zep_api_url)
zep-seeder  |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
zep-seeder  |   File "/usr/local/lib/python3.11/site-packages/zep_python/zep_client.py", line 86, in __init__
zep-seeder  |     self._healthcheck(base_url)
zep-seeder  |   File "/usr/local/lib/python3.11/site-packages/zep_python/zep_client.py", line 120, in _healthcheck
zep-seeder  |     zep_server_version = Version(zep_server_version_str.split("-")[0])
zep-seeder  |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
zep-seeder  |   File "/usr/local/lib/python3.11/site-packages/packaging/version.py", line 198, in __init__
zep-seeder  |     raise InvalidVersion(f"Invalid version: '{version}'")
zep-seeder  | packaging.version.InvalidVersion: Invalid version: ''

Thanks for reporting. We've just pushed an update to pypi.

I'm stil running into the same issue with zep-python=1.1.1
annotated-types==0.5.0
anyio==4.0.0
certifi==2023.7.22
charset-normalizer==3.2.0
exceptiongroup==1.1.3
h11==0.14.0
httpcore==0.17.3
httpx==0.24.1
idna==3.4
packaging==23.1
pydantic==2.3.0
pydantic_core==2.6.3
regex==2023.8.8
requests==2.31.0
sniffio==1.3.0
tiktoken==0.4.0
typing_extensions==4.7.1
urllib3==2.0.4
zep-python==1.1.1

Traceback (most recent call last):
  File "zep/setup/setup.py", line 9, in <module>
    client = ZepClient(base_url=zep_api_url)
  File "/Users/stephenlu/anaconda3/envs/gpt/lib/python3.8/site-packages/zep_python/zep_client.py", line 86, in __init__
    self._healthcheck(base_url)
  File "/Users/stephenlu/anaconda3/envs/gpt/lib/python3.8/site-packages/zep_python/zep_client.py", line 120, in _healthcheck
    zep_server_version = Version(zep_server_version_str.split("-")[0])
  File "/Users/stephenlu/anaconda3/envs/gpt/lib/python3.8/site-packages/packaging/version.py", line 198, in __init__
    raise InvalidVersion(f"Invalid version: '{version}'")
packaging.version.InvalidVersion: Invalid version: ''

What version of Zep server are you running? Can you share the first few lines of the Zep logs?

I suspect you're using a very old version of Zep and the newer Zep server version handling in this client was not robust to this. I've improved the server version handling in #77 and released v1.1.2.
I suggest upgrading Zep. We've added a lot more functionality :-)

Pulling the newest changes from zep-server solved the issue for me. Thanks @danielchalef !

Great!
On Fri, Sep 1 2023 at 1:57 PM, Stephen Lu < @.*** > wrote:

Pulling the newest changes from zep-server solved the issue for me. Thanks
@danielchalef ( https://github.com/danielchalef ) !
—
Reply to this email directly, view it on GitHub (
https://github.com/getzep/zep-python/issues/75#issuecomment-1703311800 ) ,
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AABAAZ4GPXP25WFX2ZQRIDDXYJD2XANCNFSM6AAAAAA4FNU3QQ
).
You are receiving this because you were mentioned. Message ID: <getzep/zep-python/issues/75/1703311800
@ github. com>


feat: add command to dump project info as JSON

there can be more added here, but it works nominally for now.
good to merge if everyone is happy.

/lgtm


feat(fetch): implement Fetch

Makes just a simple Fetch api wrapper from Motorcycle. This still needs documentation, but I just wanted to get a POC together to iterate on if needed.

This may be worth pursuing as a package maintained outside of this repository in the future.


Fixes a compile error in SourceRDD

This seems to be compiler-dependent, but I replaced it with a code snippet used in the same file in another place, which compiles fine.
Example previously failing log: https://s3.amazonaws.com/archive.travis-ci.org/jobs/171221509/log.txt
2016-10-27T21:58:42.674 [ERROR] /home/travis/build/apache/incubator-beam/runners/spark/src/main/java/org/apache/beam/runners/spark/io/SourceRDD.java:[220]
scala.collection.immutable.List.empty();
^^^^^
The method empty() is ambiguous for the type List
R: @kennknowles

Wow. I can reproduce this Travis error locally.

I updated the shade plugin, it seems to fix the ASM bug in shade plugin; tests currently running on my machine, hope it works.

Ran into the same issue with compilation of Spark, saw same fix. LGTM.

(Interestingly, the same shade plugin upgrade did not work to fix the ASM error on my machine. hmm.)
Can you tag this with BEAM-859? https://issues.apache.org/jira/browse/BEAM-859

LGTM.
This was a problem for eclipse, and the fix was offered at #1094, I forgot to apply it for Read.Unbounded implementation as well, because I don't have this problem...


vCore Resource Limits REST API

I am wondering if the vCore Resource Limits listed on this webpage are available through the REST API somewhere. The capabilities for Azure SQL Database doesn't seem to have these specs. Is there a way to return these very relevant values (Memory, IOPS, etc) via a REST API?

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

ID: b00d2236-1549-34c6-ffc4-812d9d60f662
Version Independent ID: e11d6346-7b04-4567-78c0-62d6464b6dc6
Content: Create, manage servers and single databases - Azure SQL Database
Content Source: articles/sql-database/sql-database-single-databases-manage.md
Service: sql-database
Sub-service: single-database
GitHub Login: @stevestein
Microsoft Alias: sstein

@chaneyw Thank you for the inquiry. The Azure SQL Database REST API operations do not include the capability to return service tier level settings. Can you please Up Vote and comment upon the following UserVoice entry: REST service for the enumeration of SQL Azure databases on a SQL Azure server
This feature request is under review and if you make a suggestion, it gives the PG visibility.

@chaneyw We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Include DAGs.md in documentation

Description

The catalog emits a DAGs.md file that contains a lot of great information about the DAGs. This file is a very good candidate for inclusion in the documentation site.
Implementation
It's unclear how a file that's written inside the catalog/ dir (for Docker container bind mount reasons) could be included in the docs. One approach might be symlinks but I'm unaware of how good support for symlinks is in Git and on Windows.

My goodness, the power of the monorepo!! ✨ This is a fantastic idea!
The current generate-dag-docs recipe actually moves the file once it's generated: https://github.com/WordPress/openverse/blob/75c5a68ab4016c212f1bb68c21225a857dd9f60c/catalog/justfile#L120
So we could just as easily move it under documentation/ at that step! There's no need for it to stay in the catalog/ directory 😄

My goodness, the power of the monorepo!! ✨

So much good stuff keeps appearing.

Hello @dhruvkb and @AetherUnbound,
I would like to work on this issue and I created a PR above. Please take a look and any suggestion is appreciated.


Remove version numbers from typedruby_release and typedruby_debug

Removes version numbers from typedruby_release and typedruby_debug classes.
Motivation

Including the version numbers create extra overhead while making lexer/parser changes. Such as https://github.com/sorbet/sorbet/pull/3226/files#diff-534a21ccf64c1a7b69ffc494938680a4223ef1dab50618d2a675c46095451dceR55 and similarly for Ruby 3.1 in the upcoming PR I was working on.
I don't think they bring enough value to justify the maintenance overhead and they can be removed. Thoughts?
Test plan

See included automated tests.

It also seems like we forgot to update those when pushing the 3.0 changes 🤦

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_LDEYJfhA3RWIkF
→ https://go/builds/bui_LDEYnkUd9UV2d4


fix: add litix to csp

Description
Was seeing a lot of errors for the home page video from mux

All the mux domains can be first party via *.video.thenational.academy and mux-data, mux-litix, mux-thumbnail-image so we should serve the video first party, and then the CSP can stay with the *.thenational.academy though the nested *.video. subdomain might be need to be added.

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: Support publishing from workflow dispatch tag

See: dart-lang/pub-dev#7177

It should aready work without these changes, but I removed the workflow call, as it isn't allowed to call a workflow_dispatch from non dispatch operations.

@spydon something like this?

@spydon something like this?

Yes! 😍
Just a line break missing for cellphones:

I made the image to relative width. Responsiveness is not a strength of markdown ^^

Most of the work was already done by you fortunately. I don't have write access, so feel free to merge / publish, whenever you want :)


group by function task.file.folder returns format glitch in heading when using underscores and Umlauts

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
I'm using following filter function:

```tasks
filter by function task.file.folder.includes("1_PROJECTS/")
group by function task.file.folder
not done
```

This should be return a well formated heading like (see my example in current behaviour section...)
1_PROJECTS/2024-01-02_Ä/
Current behaviour
Here's my folder / notes structure.
As you can see in the red frame there's an format glitch when using underscore and Umlauts in folders. I observed this behaviour only when there are two underscores and the last one is followed by an Umlaut.

Steps to reproduce

Create a folder whose name contains an underline (e. g. 1_PROJECTS)
Create a subfolder whose name contains an underline followed by an Umlaut (e. g. 1_PROJECTS/2024_01-01_Ä
Create a note in subfolder that contains a todo
Create a new note with following filter rules:

```tasks
filter by function task.file.folder.includes("1_PROJECTS/")
group by function task.file.folder
not done
```

Which Operating Systems are you using?

[ ] Android
[X] iPhone/iPad
[X] Linux
[X] macOS
[X] Windows

Obsidian Version
1.6.5
Tasks Plugin Version
7.6.1
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
No response

What happens when you use:
group by folder?
and what prompted to to use a custom folder instead of the built-in search?

When i use group by folder it works as expected. Just for my interest: did i something wrong or shouldn't group by function task.file.folder the same thing?

The group by root|folder|path built-in commands all take care of escaping specials characters in the path, to fix the problem you identified.
The filter by function instructions are provided for handling cases that the built-in commands cannot do, via things like string manipulation. In that case, it's up to the user to do any escaping, since of course Tasks has no idea what the user intended to do.
The reason I asked the following ...

and what prompted you to use a custom folder instead of the built-in search?

... is that there has been a sudden increase in users asking for help with filter|sort|group by function to do things that the built-in commands do perfectly well. And I am wondering why...

I've made a small change to the documentation to make visible the difference between this and similar built-in and custom grouping commands: aec79e04427d85839e5cbf804693fa3ff7577ade.

The group by root|folder|path built-in commands all take care of escaping specials characters in the path, to fix the problem you identified.
The filter by function instructions are provided for handling cases that the built-in commands cannot do, via things like string manipulation. In that case, it's up to the user to do any escaping, since of course Tasks has no idea what the user intended to do.

Thank you very much for your explanation.

The reason I asked the following ...

and what prompted you to use a custom folder instead of the built-in search?

... is that there has been a sudden increase in users asking for help with filter|sort|group by function to do things that the built-in commands do perfectly well. And I am wondering why...

In my case i think it's simply a lack of knowledge in combination with some time pressure. Before i came across the filter function i was searching for a quick resolution that shows me all open todos in my projects. So i skimmed over the Obsidian Tasks documentation and found the grouping section, without diving deeper to understand how the grouping function really works. At the beginning group by function task.file.folder worked for me, until i used an Umlaut the first time...
I saw you changed the documentation a little bit and that's very helpful. It's interesting, because i actually thought there must be something wrong with group by function task.file.folder because of the wording "Same as 'group by folder'."
I think your change of the documentation helps people like me, that are only skimming over the documentation to find a tool that quickly solves the problem they have. I hope this helps other people too so you're getting less bug reports.
I would like to thank you very much. I saw you're doing a lot of work to make this software better and better. And at the same time you take the time to take care of the community who don't have such a deep understanding. Thank's for that ;-)

Thanks for the kind words and feedback.
I'll close this now, as no action is required.


Fixing the typo as listed in ticket 41628

Summary
I fixed the typo as listed in the ticket.
Fixes #41628

Internal previews

📄 File
🔗 Preview link

docs/csharp/language-reference/keywords/method-parameters.md
Method Parameters

@BillWagner thanks for the response! i'll look at writing some examples for this per your comment in the original issue, and i'll also look at that help-wanted label for further issues.


update discord.js

we're skipping this version since v14 is already out. just waiting for the next discord-player update to get merged


OF: t2.small ec2 instances are not available in us-west-2d

Motivation
During benchmark testing of Pulsar with openmessaging/benchmark I noticed that t2.small ec2 instances are not available in us-west-2d.
Modifications
Changed t2.small to t3.small in three places in two files.
Verifying this change

[x] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Anything that affects deployment: (yes)

Documentation

[X] doc

Hi @nodece can you review this PR from the technical perspective? Thanks

@dave2wave Thanks for your contribution!

@Anonymitaet This PR can be merged.


ci: ci/release support macos arm

use macos-11 to run ci/release

@killagu , a new test job has been submitted. Please wait in patience.

@killagu , your pull request has been updated, but no change is detected in the code.

release workflow has been test. https://github.com/cnpm/image-service/runs/7277140716?check_suite_focus=true

@killagu , The CI test is completed, please check result:

Test CaseTest Result

merge-target-branch:white_check_mark:SUCCESS

build-docker-image:white_check_mark:SUCCESS

compile-nydus:white_check_mark:SUCCESS

compile-ctr-remote:white_check_mark:SUCCESS

compile-nydus-snapshotter:white_check_mark:SUCCESS

start-nydus-snapshotter-config-containerd:white_check_mark:SUCCESS

run-container-with-nydus-image:white_check_mark:SUCCESS

Congratulations, your test job passed!


Fixing the contrast and centering in user greeting

To do:-

[ ] Change the user greeting text color to white.
[ ] Center the user greeting in both full screen and mobile version.

@Vividh25 , Can I work on this one?

@Vividh25 , Can I work on this one?

Assigned. I have updated the issue. Do have a look once.


Refactor ReportDiagnosticMessage to use a separate Logger class with virtual functions

View progress in https://github.com/Deadpikle/NetSparkle/tree/feature/log

This feature is now complete.


Stuck on Install package when calling gvm use

Hey there! I'm a newbie to gvm, wanna get some help ❤
I installed go1.3 with gvm install go1.3 --binary, then when I want to activate my environment, it always requires me to install two packages, blocking the input:
08:13 $ gvm use go1.3
Now using version go1.3
Install package 'git-core' to provide command 'git'? [N/y] Install package 'autojump' to provide command 'autojump'? [N/y]
# I can't input anything.

Maybe there are some conflict in my bashrc, cause when I start from a clean bash, gvm just works file.
bash-5.0$ [[ -s "/home/ofey/.gvm/scripts/gvm" ]] && source "/home/ofey/.gvm/scripts/gvm"
bash-5.0$ gvm use go1.3
Now using version go1.3
bash-5.0$ go version
go version go1.3 linux/amd64

Here is my bashrc.
I commented two suspicious line:
# [[ $- == *i* ]] && source /opt/bash-git-prompt/gitprompt.sh
# [[ -f /etc/profile.d/autojump.sh ]] && source /etc/profile.d/autojump.sh

Then I can input, but it just keeps bursting some error messages. Besides, some builtin commands like grep is lost.(Cause my $PATH is gone?)
[ofey@RX93 dotfiles]$ gvm use go1.3
Now using version go1.3
[ofey@RX93 dotfiles]$ Install package 'autojump' to provide command 'autojump'? [N/y]

[ofey@RX93 dotfiles]$ Install package 'autojump' to provide command 'autojump'? [N/y]
grep
bash: grep: command not found...
Install package 'grep' to provide command 'grep'? [N/y] n

[ble: exit 127]
[ofey@RX93 dotfiles]$ Install package 'autojump' to provide command 'autojump'? [N/y]

[ofey@RX93 dotfiles]$ Install package 'autojump' to provide command 'autojump'? [N/y]
echo $PATH
/home/ofey/.gvm/pkgsets/go1.3/global/bin:/home/ofey/.gvm/gos/go1.3/bin:/home/ofey/.gvm/pkgsets/go1.3/global/overlay/bin:/home/ofey/.gvm/bin:/home/ofey/.gvm/bin:Error: : is not a valid task number!
[ofey@RX93 dotfiles]$ Install package 'autojump' to provide command 'autojump'? [N/y]

Thank you for your attention! ⭐

any update?

It's fine to close this issue, I have no progress since then ;)