U++ framework
Do not panic. Ask here before giving up.

Home » U++ TheIDE » U++ TheIDE: Other Features Wishlist and/or Bugs » [PROPOSAL] Configurable web search menu for TheIDE
[PROPOSAL] Configurable web search menu for TheIDE [message #55394] Fri, 06 November 2020 11:34 Go to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Hi,

I've been using TheIDE with this configurable web search menu for some time and I find it very useful, so I thought it might be a good idea to propose it for the trunk.

A direct search on web sites sometimes yield better results than using a generic search engine such as google. E.g. just search for "private" on Google and CppReference and see the results. Smile

What I propose is:
* Adding an optional sub-menu to TheIDE's context menu. This menu will become visible only if a search provider is set.
* Adding a simple search provider setup tab in the setting menu.

- This proposal does not change Google and Upp website search entries, and leave them intact (as default, hard-coded options).

Screenshots:

https://i.imgur.com/Mazz8oG.png

https://i.imgur.com/yqABDpX.png


What do you think?


Best regards,
Oblivion






[Updated on: Fri, 06 November 2020 11:38]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55396 is a reply to message #55394] Fri, 06 November 2020 15:14 Go to previous messageGo to next message
Didier is currently offline  Didier
Messages: 740
Registered: November 2008
Location: France
Contributor
Seems to be a good idea to me.
Even a better one if it's already developped Wink
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55397 is a reply to message #55396] Fri, 06 November 2020 16:06 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Hello Didier,

Quote:
Seems to be a good idea to me.
Even a better one if it's already developped


It is already developed, yes. Smile To be honest, it is a simple string map manager, nothing more.
Easy to maintain.

Of course, the setup dialog may need a bit polishing.

If Mirek greenlight's this, I will send in the patches for review and modification.

Best regards,
Oblivion


[Updated on: Fri, 06 November 2020 16:07]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55398 is a reply to message #55397] Sat, 07 November 2020 05:43 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
We might want to have means to define "default" search engine for Alt+G....
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55400 is a reply to message #55398] Sat, 07 November 2020 09:23 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Quote:
We might want to have means to define "default" search engine for Alt+G....


OK, I'll add that to the mechanism.

Still I think we shouldn'T add Google and Upp-website on the configurable list. They should be hard-coded.

And how should we handle configuration?

Currently I use binary serialization, but it is far from ideal.
For trunk, I suggest we use a separate file ("ide.websearch" or "ide.search") with JSON or other format

Best regards,
Oblivion


[Updated on: Sat, 07 November 2020 09:25]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55423 is a reply to message #55398] Mon, 09 November 2020 21:36 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Hello Mirek,

Please find attached the patch to be reviewed for the proposed online search engines support.

This patch:

+ Adds a search providers manager tab to settings dialog.
+ Adds the ability to set a default search provider.
+ Adds the ability to sort the search providers menu.
+ Uses a separate, JSON formatted config file ("ide.search") to store the search configuration.


If you think it needs improvements, let me know.

Best regards,
Oblivion


[Updated on: Mon, 09 November 2020 21:36]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55426 is a reply to message #55398] Tue, 10 November 2020 11:08 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Hello Mirek,

I improved and polished the patch.

Now the search providers menu will be hidden if the search providers list has a single entry that is set as the defeault provider.

Best regards,
Oblivion



[Updated on: Tue, 10 November 2020 11:17]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55427 is a reply to message #55400] Tue, 10 November 2020 11:19 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Oblivion wrote on Sat, 07 November 2020 09:23
Quote:
We might want to have means to define "default" search engine for Alt+G....


Still I think we shouldn'T add Google and Upp-website on the configurable list. They should be hard-coded.



Actually, if U++ ever started to be used more widely, nice possible source of funding there... Smile

Mirek
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55445 is a reply to message #55427] Wed, 11 November 2020 10:15 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Quote:
Actually, if U++ ever started to be used more widely, nice possible source of funding there...


Fingers crossed. Smile

Anyway, the above patch adds the ability to set ALT + G and default search provider.
In time if Upp gains more audience, we can simply ship a selected set of default entries too (Json file compiled-in as a binary resource, maybe). For now, the hard-coded default is Google but it can be changed by users, using the web search manager or simply by editing the ide.search file.

The real benefit of this configurable search engine is that it allows users to do something like this:

index.php?t=getfile&id=6278&private=0

Smile

Best regards,
Oblivion


[Updated on: Wed, 11 November 2020 10:17]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55446 is a reply to message #55445] Wed, 11 November 2020 14:41 Go to previous messageGo to next message
Klugier is currently offline  Klugier
Messages: 1118
Registered: September 2012
Location: Poland, Kraków
Senior Contributor
Hello,

I would personally simplify editor menu. Too many options there is not good option and might be misleading. So I opt for moving "Search on Google" and "Search on the official site.." to "Search on menu" (This entry shouldn't have .., because it doesn't spawn any external dialog or redirect to browser):
Search on ->
   Google..
   Official site..
   Separator /* Not hard coded engines */
   CppReference..
   DuckDuckGo..
   ...

Thanks to that editor menu will not be overload with options.

Also "ide.search" nice idea for external modification, however we should use regular TheIDE serialization procedures (binary) and bump settings version. We should keep all TheIDE options in standardize way. If we want to keep it in external file that is easy to modify by the user then let's do it for all options. Partial solution will always increase fragmentation.

I like the idea of supporting more sources. With the solution that we will have the option to specify default engine only we will lose that ability. Also, sites like CppReference is extremely useful for the learning purpose (C++ and standard library).

If we will gain popularity - sure we could try selling default option here. Maybe we should ask DuckDuckGo if they want to buy it Smile

Klugier


U++ - one framework to rule them all.

[Updated on: Wed, 11 November 2020 16:12]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55447 is a reply to message #55446] Wed, 11 November 2020 16:22 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Hello Klugier,

Quote:
would personally simplify editor menu. Too many options there is not good option and might be misleading. So I opt for moving "Search on Google" and "Search on the official site.." to "Search on menu"


OK.


Quote:
however we should use regular TheIDE serialization procedures (binary) and bump settings version


I was hesitant to add a Serialize method (not to disrupt TheIDE's config settings), but this sounds OK to me. Still, I'd prefer an external file in human-readble form (Json seems very good to me. besides it can allow us to easily change/add new stuff if need be).

I'll upload a patch with the changes you've asked, later tonight.

It is up to Mirek to decide.

p.s: Speaking of consistency, I think TheIDE's toolbar should have an Icon for "Settings" menu entry.

Best regards,
Oblivion


[Updated on: Wed, 11 November 2020 16:25]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55454 is a reply to message #55447] Thu, 12 November 2020 20:31 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Oblivion wrote on Wed, 11 November 2020 16:22
Hello Klugier,

Quote:
would personally simplify editor menu. Too many options there is not good option and might be misleading. So I opt for moving "Search on Google" and "Search on the official site.." to "Search on menu"


OK.


Quote:
however we should use regular TheIDE serialization procedures (binary) and bump settings version


I was hesitant to add a Serialize method (not to disrupt TheIDE's config settings), but this sounds OK to me. Still, I'd prefer an external file in human-readble form (Json seems very good to me. besides it can allow us to easily change/add new stuff if need be).


Agreed. JSON is a bit off (we mostly have key=value pairs), but it is ok.

Quote:

p.s: Speaking of consistency, I think TheIDE's toolbar should have an Icon for "Settings" menu entry.


Do not quite agree - if it is settings, it should not be needed that often.

Mirek
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55455 is a reply to message #55447] Thu, 12 November 2020 20:32 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Oblivion wrote on Wed, 11 November 2020 16:22

I'll upload a patch with the changes you've asked, later tonight.


Ops, I have just patched... I will take it from here.

Mirek
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55456 is a reply to message #55455] Thu, 12 November 2020 20:46 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
I am sorry but the code feels a bit overengineered. There will be simplifications Smile

That said, do the phrase placeholder need to be so complicated? Would not simple '@' be enough?

More importantly, we need a default list.

Mirek
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55457 is a reply to message #55456] Thu, 12 November 2020 21:02 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Quote:
do the phrase placeholder need to be so complicated? Would not simple '@' be enough?


Sure, @ would do just fine.
I was following KDE/Konqueror's style there. Gnome web uses "%s", for example.

Quote:

I am sorry but the code feels a bit overengineered. There will be simplifications


By all means, go on. Smile

Quote:
More importantly, we need a default list.


Yes, I was going to propose that too, but apparently I am a bit late.

Still I think, at least Google, Upp, CppReference, StackOverflow [+Github] could be in the list.
Thanks.


Best regards,
Oblivion


[Updated on: Thu, 12 November 2020 21:04]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55471 is a reply to message #55457] Sat, 14 November 2020 13:07 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Thanks for comments even if the issue is not yet finished.

WRT ReadIcon... Well, I have found the ReadIcon code lacking, but that was before the commit, so should have been fixed there. BTW, the issue with cppreference is that they do not return correct .ico format, they return regular .png instead with .ico extension Smile And ReadIcon crashed on png. But it does not anymore when I am testing. Can you check this one?

GUI_APP_MAIN
{
	String icon = HttpRequest("https://en.cppreference.com/favicon.ico").Execute();
	for(const Image& m : ReadIcon(icon))
		DDUMP(m.GetSize());
}


If you have the actual ReadIcon version and it is still crashing/freezing, can you send me a backtrace?

As for the default list, StackOverflow has captcha, not sure we want that...

Mirek
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55472 is a reply to message #55471] Sat, 14 November 2020 13:58 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Hello Mirek,


Quote:
ReadIcon crashed on png. But it does not anymore when I am testing. Can you check this one?


I've just synced my copy of trunk and recompiled TheIDE (on Linux, in both DEBUG and RELEASE modes). The issue seems to be resolved. The search manager downloaded and displayed the icons correctly. Thanks!


By the way, some more observations on the new code:

1. It does not discard empty entries in json file (while reading). Empty entries can show up in both menu and in the search manager. The original patch prevented this. A cosmetics issue really, but looks ugly when it happens.

2. Default Google search is not available when the list has at least one entry. (The original patch prevented this too by allowing the user to unset the default engine) The only way to reset the default search engine to Google is by deleting the entire list.

3. ArrayCtrl has ScrollBar visible. Shouldn't it be AutoHideScrollbar() ?

4. ArrayCtrl has visible vertical and horizontal grids, which seems unnecessary as it uses even/odd line coloring already.

As for the StackOverflow issue,

I have now checked and yes, you are right. We should find anothe provider.

I'd recommend Microsoft Docs (for at least Windows builds)



Thank you for your efforts.

Best regards,
Oblivion


[Updated on: Sat, 14 November 2020 14:05]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55476 is a reply to message #55394] Sat, 14 November 2020 14:46 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
I am finished for now. Please check.
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55477 is a reply to message #55472] Sat, 14 November 2020 14:51 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Oblivion wrote on Sat, 14 November 2020 13:58
Hello Mirek,


Quote:
ReadIcon crashed on png. But it does not anymore when I am testing. Can you check this one?


I've just synced my copy of trunk and recompiled TheIDE (on Linux, in both DEBUG and RELEASE modes). The issue seems to be resolved. The search manager downloaded and displayed the icons correctly. Thanks!


By the way, some more observations on the new code:

1. It does not discard empty entries in json file (while reading). Empty entries can show up in both menu and in the search manager. The
original patch prevented this. A cosmetics issue really, but looks ugly when it happens.


I guess the only way how this can happen is if user manually edits .json. Is it worth fixing?

Quote:

2. Default Google search is not available when the list has at least one entry. (The original patch prevented this too by allowing the user to unset the default engine) The only way to reset the default search engine to Google is by deleting the entire list.


Or set first entry as google... I now expect to that "in code" default search only be active if the list is missing.

Quote:

4. ArrayCtrl has visible vertical and horizontal grids, which seems unnecessary as it uses even/odd line coloring already.


I do not like that when list has columns.

Quote:

Thank you for your efforts.


Ah that was your initiative! Good job, this is a nice feature!

Mirek
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55478 is a reply to message #55477] Sat, 14 November 2020 15:27 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Everything is up and working well here.

Quote:
I guess the only way how this can happen is if user manually edits .json. Is it worth fixing?


From a personal POV, I don't really care. I do not edit a config file manually unless I necessarily have to. But I know and work with a plenty of *NIX users who almost "live" in those files and who consider using a GUI for it a heresy. Smile

From a QA perspective, it'll give a better impression on a production-level code.

Still, it is not really a must. You decide.

Quote:
I do not like that when list has columns.


OK, I've noted down this too (for possible future patches). Smile


Again, thank you and have a nice weekend (or what's left of it).

Best regards,
Oblivion


[Updated on: Sat, 14 November 2020 15:33]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55479 is a reply to message #55476] Sat, 14 November 2020 17:09 Go to previous messageGo to next message
Novo is currently offline  Novo
Messages: 1431
Registered: December 2006
Ultimate Contributor
mirek wrote on Sat, 14 November 2020 08:46
I am finished for now. Please check.

Compilation with Mac and Clang-MinGW is broken.
./umk uppsrc ide CLANG -bu
/home/buildbot/worker/mingw-upp/build/uppsrc/Core/SSL/SSL.h:3:10: fatal error: 'openssl/ssl.h' file not found
#include <openssl/ssl.h>
         ^~~~~~~~~~~~~~~
1 error generated.


Regards,
Novo

[Updated on: Sat, 14 November 2020 17:13]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55480 is a reply to message #55479] Sat, 14 November 2020 18:48 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Novo wrote on Sat, 14 November 2020 17:09
mirek wrote on Sat, 14 November 2020 08:46
I am finished for now. Please check.

Compilation with Mac and Clang-MinGW is broken.
./umk uppsrc ide CLANG -bu
/home/buildbot/worker/mingw-upp/build/uppsrc/Core/SSL/SSL.h:3:10: fatal error: 'openssl/ssl.h' file not found
#include <openssl/ssl.h>
         ^~~~~~~~~~~~~~~
1 error generated.


Not sure what Clang-MinGW is. With LLVM-MinGW as supplied with Win32 version and with all libraries, it works.

MacOS builds too. You need openssl installed. Which I agree is a bummer, but I do not see any other way. Alternative is not to have favicons loading in MacOS, but then the need for openssl will probably hit us later anyway (e.g. with uppiverse). So I am afraid we will have it as prerequisite from now on Sad

Mirek

[Updated on: Sat, 14 November 2020 19:08]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55482 is a reply to message #55480] Sat, 14 November 2020 20:39 Go to previous messageGo to next message
Klugier is currently offline  Klugier
Messages: 1118
Registered: September 2012
Location: Poland, Kraków
Senior Contributor
Hello,

I would like to add that I observe following problem:
- When the list is empty all buttons are enable. They should be disable

I just wonder if we could integrate the buttons with the array, I think the would look better. And why they are placed from right to left and not from right to left like toolbar Smile

Restart to defaults would be handy option If someone mess there. For example I was on the previous version and when updating to current default engines are not there. This is true for all settings!

Klugier


U++ - one framework to rule them all.

[Updated on: Sat, 14 November 2020 20:49]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55484 is a reply to message #55482] Sun, 15 November 2020 00:09 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Klugier wrote on Sat, 14 November 2020 20:39
Hello,

I would like to add that I observe following problem:
- When the list is empty all buttons are enable. They should be disable


Absolutely.

Quote:

I just wonder if we could integrate the buttons with the array, I think the would look better. And why they are placed from right to left and not from right to left like toolbar Smile


I wanted the same design as in nest manager. Actualy, I copied button layout from there...

Anyway, I believe button really look better placed from right to left. I have not seen a dialog that would have it otherwise.



Quote:

Restart to defaults would be handy option If someone mess there. For example I was on the previous version and when updating to current default engines are not there. This is true for all settings!


It is there, just only in the context menu.

Mirek

[Updated on: Sun, 15 November 2020 00:09]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55486 is a reply to message #55482] Sun, 15 November 2020 00:57 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Klugier wrote on Sat, 14 November 2020 20:39
Hello,

I would like to add that I observe following problem:
- When the list is empty all buttons are enable. They should be disable


Fixed. Sorry, just forgot about it...

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55493 is a reply to message #55480] Sun, 15 November 2020 17:10 Go to previous messageGo to next message
Novo is currently offline  Novo
Messages: 1431
Registered: December 2006
Ultimate Contributor
mirek wrote on Sat, 14 November 2020 12:48
You need openssl installed.

Sorry, my bad. I forgot that my build automation system for Mac and Windows didn't have openssh dev-packages installed.


Regards,
Novo
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55515 is a reply to message #55480] Wed, 18 November 2020 21:36 Go to previous messageGo to next message
Novo is currently offline  Novo
Messages: 1431
Registered: December 2006
Ultimate Contributor
mirek wrote on Sat, 14 November 2020 12:48

MacOS builds too. You need openssl installed. Which I agree is a bummer, but I do not see any other way. Alternative is not to have favicons loading in MacOS, but then the need for openssl will probably hit us later anyway (e.g. with uppiverse). So I am afraid we will have it as prerequisite from now on Sad

I've installed OpenSSL on Mac, but I'm getting this linking error:
./umk reference SignPDF CLANG -bus

Linking...
Undefined symbols for architecture x86_64:
  "Upp::GetAllFacesSys()", referenced from:
      Upp::Font::FaceList() in Draw$blitz.o
  "Upp::GetFontDataSys(Upp::Font)", referenced from:
      Upp::Font::GetData() const in Draw$blitz.o
  "Upp::GetFontInfoSys(Upp::Font)", referenced from:
      Upp::GetFontInfo(Upp::Font) in Draw$blitz.o
  "Upp::GetGlyphInfoSys(Upp::Font, int)", referenced from:
      Upp::IsNormal(Upp::Font, int) in Draw$blitz.o
      Upp::GetGlyphEntry(Upp::Font, int, unsigned long long) in Draw$blitz.o
  "Upp::RenderCharacterSys(Upp::FontGlyphConsumer&, double, double, int, Upp::Font)", referenced from:
      Upp::Font::Render(Upp::FontGlyphConsumer&, double, double, int) const in Draw$blitz.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)


Regards,
Novo
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55516 is a reply to message #55515] Wed, 18 November 2020 22:00 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Novo wrote on Wed, 18 November 2020 21:36
mirek wrote on Sat, 14 November 2020 12:48

MacOS builds too. You need openssl installed. Which I agree is a bummer, but I do not see any other way. Alternative is not to have favicons loading in MacOS, but then the need for openssl will probably hit us later anyway (e.g. with uppiverse). So I am afraid we will have it as prerequisite from now on Sad

I've installed OpenSSL on Mac, but I'm getting this linking error:
./umk reference SignPDF CLANG -bus

Linking...
Undefined symbols for architecture x86_64:
  "Upp::GetAllFacesSys()", referenced from:
      Upp::Font::FaceList() in Draw$blitz.o
  "Upp::GetFontDataSys(Upp::Font)", referenced from:
      Upp::Font::GetData() const in Draw$blitz.o
  "Upp::GetFontInfoSys(Upp::Font)", referenced from:
      Upp::GetFontInfo(Upp::Font) in Draw$blitz.o
  "Upp::GetGlyphInfoSys(Upp::Font, int)", referenced from:
      Upp::IsNormal(Upp::Font, int) in Draw$blitz.o
      Upp::GetGlyphEntry(Upp::Font, int, unsigned long long) in Draw$blitz.o
  "Upp::RenderCharacterSys(Upp::FontGlyphConsumer&, double, double, int, Upp::Font)", referenced from:
      Upp::Font::Render(Upp::FontGlyphConsumer&, double, double, int) const in Draw$blitz.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)


Well, I am actually starting to think it might be better to disable this on macos...
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55517 is a reply to message #55516] Wed, 18 November 2020 22:16 Go to previous messageGo to next message
Novo is currently offline  Novo
Messages: 1431
Registered: December 2006
Ultimate Contributor
mirek wrote on Wed, 18 November 2020 16:00

Well, I am actually starting to think it might be better to disable this on macos...

Disable testing of SignPDF on Mac, or disable OpenSSL on Mac completely?


Regards,
Novo
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55518 is a reply to message #55517] Thu, 19 November 2020 05:22 Go to previous messageGo to next message
Novo is currently offline  Novo
Messages: 1431
Registered: December 2006
Ultimate Contributor
Actually, SignPDF is the only app having problem with compilation on Mac.
IMHO, it makes sense to fix it.


Regards,
Novo
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55537 is a reply to message #55518] Fri, 20 November 2020 14:23 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Novo wrote on Thu, 19 November 2020 05:22
Actually, SignPDF is the only app having problem with compilation on Mac.
IMHO, it makes sense to fix it.


Should be now fixed.
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55538 is a reply to message #55537] Fri, 20 November 2020 14:47 Go to previous messageGo to next message
Novo is currently offline  Novo
Messages: 1431
Registered: December 2006
Ultimate Contributor
mirek wrote on Fri, 20 November 2020 08:23
Novo wrote on Thu, 19 November 2020 05:22
Actually, SignPDF is the only app having problem with compilation on Mac.
IMHO, it makes sense to fix it.


Should be now fixed.

./umk uppsrc umk CLANG -bus

duplicate symbol __ZN3Upp14GetAllFacesSysEv in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
duplicate symbol __ZN3Upp15GetGlyphInfoSysENS_4FontEi in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
duplicate symbol __ZN3Upp18RenderCharacterSysERNS_17FontGlyphConsumerEddiNS_4FontE in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
duplicate symbol __ZN3Upp14GetFontInfoSysENS_4FontE in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
duplicate symbol __ZN3Upp14GetFontDataSysENS_4FontE in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
ld: 5 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)


Regards,
Novo
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55539 is a reply to message #55538] Fri, 20 November 2020 15:03 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Novo wrote on Fri, 20 November 2020 14:47
mirek wrote on Fri, 20 November 2020 08:23
Novo wrote on Thu, 19 November 2020 05:22
Actually, SignPDF is the only app having problem with compilation on Mac.
IMHO, it makes sense to fix it.


Should be now fixed.

./umk uppsrc umk CLANG -bus

duplicate symbol __ZN3Upp14GetAllFacesSysEv in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
duplicate symbol __ZN3Upp15GetGlyphInfoSysENS_4FontEi in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
duplicate symbol __ZN3Upp18RenderCharacterSysERNS_17FontGlyphConsumerEddiNS_4FontE in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
duplicate symbol __ZN3Upp14GetFontInfoSysENS_4FontE in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
duplicate symbol __ZN3Upp14GetFontDataSysENS_4FontE in:
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/umk/CLANG.Debug.Debug_Full.Main.Shared/umk$blitz.o
    /Users/ssg/.local/soft/bb-worker/worker/m-upp/build/.cache/upp.out/Draw/CLANG.Debug.Debug_Full.Shared/FontCoco.o
ld: 5 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)


Fix one break another? Smile

Mirek
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55540 is a reply to message #55539] Fri, 20 November 2020 15:09 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Hopefully fixed. (Actually, fixed by removing the previous fix for macos Smile
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55541 is a reply to message #55540] Fri, 20 November 2020 15:50 Go to previous messageGo to next message
Novo is currently offline  Novo
Messages: 1431
Registered: December 2006
Ultimate Contributor
mirek wrote on Fri, 20 November 2020 09:09
Hopefully fixed. (Actually, fixed by removing the previous fix for macos Smile

Everything is fine now, including SignPDF. Smile
Thanks!


Regards,
Novo
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55620 is a reply to message #55394] Tue, 24 November 2020 12:30 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Hello Mirek,

The default Google entry does not work on Linux:

It should be:

{ "Google", "https://www.google.com/search?q=%s" },
             -----                  ------



Best regards,
Oblivion


Re: [PROPOSAL] Configurable web search menu for TheIDE [message #55622 is a reply to message #55620] Tue, 24 November 2020 12:49 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Thanks, fixed.
Re: [PROPOSAL] Configurable web search menu for TheIDE [message #56506 is a reply to message #55622] Fri, 19 March 2021 20:33 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Hello Mirek,

A small cosmetics issue: the "Move down" item of the online search providers list context menu has the wrong icon.


https://github.com/ultimatepp/ultimatepp/blob/6eec1d97948106 4d31616125769f5a5b06fc9a81/uppsrc/ide/OnlineSearch.cpp#L89

bar.Add(b, "Move down", IdeImg::arrow_up(), [=] { list.SwapDown(); Sync(); }).Key(K_CTRL_DOWN);
                                ----------
                                   ^^^



Best regards,
Oblivion


[Updated on: Fri, 19 March 2021 20:35]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #56524 is a reply to message #56506] Sun, 21 March 2021 13:31 Go to previous messageGo to next message
Klugier is currently offline  Klugier
Messages: 1118
Registered: September 2012
Location: Poland, Kraków
Senior Contributor
Hello Oblivion,

The fix is on the trunk. Should be available on tomorrows nightly.

Klugier


U++ - one framework to rule them all.

[Updated on: Sun, 21 March 2021 13:32]

Report message to a moderator

Re: [PROPOSAL] Configurable web search menu for TheIDE [message #56534 is a reply to message #56524] Mon, 22 March 2021 11:06 Go to previous message
Oblivion is currently offline  Oblivion
Messages: 1266
Registered: August 2007
Senior Contributor
Quote:
Hello Oblivion,

The fix is on the trunk. Should be available on tomorrows nightly.


Thanks, it is fixed.


Best regards,
Oblivion


Previous Topic: Displaying screenshots in Readme (UppHub)
Next Topic: Import external library - Include files depending on OS
Goto Forum:
  


Current Time: Sat May 23 16:44:52 GMT+2 2026

Total time taken to generate the page: 0.01884 seconds