Skip to content

Adicionado no configuration manager para aceitar compilação x64 é x86 - #150

Merged
adeniltonbs merged 4 commits into
ZeusAutomacao:masterfrom
robertorp:master
Jul 4, 2016
Merged

Adicionado no configuration manager para aceitar compilação x64 é x86#150
adeniltonbs merged 4 commits into
ZeusAutomacao:masterfrom
robertorp:master

Conversation

@robertorp

Copy link
Copy Markdown
Contributor

Adicionado Nuget
Quando coloquei no configuration manager para aceitar compilação de x64 é x86 ele criou sozinho essas milhões de linhas.

@adeniltonbs bom se quiser os dados do nuget o repositório e somente enviar um e-mail para min que lhe passo o mesmo. robertoalvespereira18@gmail.com

Bom se pesqusiar por "zeus nfe" irá achar dois repositórios onde um é x86 e o outro x64 o x64 tem .64 na frente.

@rftd

rftd commented Jun 25, 2016

Copy link
Copy Markdown

Só não entendi o por qque de fazer x86 e x64 afinal o AnyCpu server para os dois.
Isto é totalmente desnecessário em código Managed.

@robertorp

Copy link
Copy Markdown
Contributor Author

Se eu compilar numa maquina x64(anycpu) e usar a dll no x86 (pc) ele irá rodar normalmente? (não sabia haha) e porque aqui nosso projeto e todo feito x86 e me acostumei assim

@rftd

rftd commented Jun 25, 2016

Copy link
Copy Markdown

sim o AnyCpu roda em qq maquina, sem a maqui for x86 o programa roda em x86 se for x64 ele roda em x64.
Geralmente especificamos o tipo quando tem que usar dll nativa por exemplo no caso do SAT maioria dos fabricantes compilam a dll so em x86 ai o seu programa precisa ser compilado tbm em x86.
Ai mesmo se a dll for AnyCpu e o pc x64 ela vai funcionar em x86 pois o programa que esta executando ela é x86.

@adeniltonbs

Copy link
Copy Markdown
Member

Melhor deixar AnyCpu então. Vc pode alterar isso Roberto? Outra coisa, ainda não li sobre o funcionamento do nuget. Mas é realmente necessário versionar a pasta NFe.AppTeste\obj* ?

@robertorp

Copy link
Copy Markdown
Contributor Author

De acordo com o @rftd não precisa, vou estar arrumando, amanhã anoite.
Ai eu tiro o build que coloquei no nuget x64 e deixo somente o AnyCpu.

@adeniltonbs

Copy link
Copy Markdown
Member

👍

@robertorp

robertorp commented Jun 28, 2016

Copy link
Copy Markdown
Contributor Author

conforme conversado ajustei ele novamente, deletei o x86 é x64

<metadata>
<id>Zeus.Net.NFe.NFCe</id>
<version>1.0.1.154</version>
<title>Zeus NF-e é NFC-e (x86/x64)</title>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Este "é" está com acento de forma incorreta acho

Adicionado nuget no readme
@robertorp

Copy link
Copy Markdown
Contributor Author

@adeniltonbs confere, se tiver alguma outra coisa e somente dizer

@rftd

rftd commented Jun 28, 2016

Copy link
Copy Markdown

Boa tarde o nupkg não precisa add so o nuspec.
No nuget não adicione a dll de impressão pois a mesma precisa do FastReport o ideal seria criar um package a parte para ela.

@adeniltonbs

Copy link
Copy Markdown
Member

@robertorp, como o Rafael sugeriu, poderia remover a dll NFe.Impressao do pacote, daí aceito o pull. Obrigado desde já pela colaboração.

@adeniltonbs

adeniltonbs commented Jun 29, 2016

Copy link
Copy Markdown
Member

Essa semana ainda removo a dll NFe.Impressao da solução e coloco em outra.

@adeniltonbs

Copy link
Copy Markdown
Member

Lá no nuget o "e" ainda está acentuado
image

@robertorp

Copy link
Copy Markdown
Contributor Author

Ajustado

@adeniltonbs
adeniltonbs merged commit a66f0f3 into ZeusAutomacao:master Jul 4, 2016
AgnaldoSilva0 pushed a commit to AgnaldoSilva0/DFe.NET that referenced this pull request Aug 12, 2026
AgnaldoSilva0 pushed a commit to AgnaldoSilva0/DFe.NET that referenced this pull request Aug 12, 2026
AgnaldoSilva0 added a commit to AgnaldoSilva0/DFe.NET that referenced this pull request Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants