Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filer crash immediately #95

Closed
Giova84 opened this issue Aug 2, 2017 · 24 comments
Closed

Filer crash immediately #95

Giova84 opened this issue Aug 2, 2017 · 24 comments

Comments

@Giova84
Copy link

Giova84 commented Aug 2, 2017

I've built filer from the last available commit 514d5dd

All went fine, but when I launch Filer, it will immediately crash.
I saved the debug report: http://sprunge.us/gQFA

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Please notice that I attempted to build it (Filer and Autofiler) by using GCC2 and GCC5.

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Sorry for the noise. I removed the AutoFilerFolders file (from 15 July) and now Filer starts correctly. But now, when I attempt to add folders to filer, Autofiler will crash. And then, when I quit Filer and start it again, it will crash again as before.
To start it again, I have to remove again the AutoFilerFolders file. But in facts, if I add folders to watch, AutoFiler will crash and I have to delete the AutoFilerFolders file.

Debug report for Autofiler: http://sprunge.us/XNNI

@humdingerb
Copy link
Member

Thanks for the report!

This is the culprit

@owenca: reverting to folderList->AddItem(new BStringItem(str)); fixes it.

@Giova84: Can I interest you in doing an Italian translation, see #92 ? :)

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Hi Humdinger, I already have a it.catkeys file (which I made today) I'd like to test it on Filer, to see if everything will look fine :-) Where I will have to place this file to see Filer translated?
I'll wait for this ticket to be fixed to build the new version, I'll check the translation inside the app and then I will share the catkey file with you :-)

@humdingerb
Copy link
Member

You can revert Filer's source to before the breakage with:
git reset --hard f2d805d91a2732d508d1a8f99e5dd2dd128cccda

  • Put your "it.catkeys" into the sources/locale/ folder
  • Open the Makefile and at "it" at LOCALES
  • Do a "make" followed by "make bindcatalogs"

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

After git reset --hard f2d805d91a2732d508d1a8f99e5dd2dd128cccda
I got:

Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set)```

I obviously miss something because I admit that I am poor with GIT :-)

@humdingerb
Copy link
Member

Sorry, I'm not familiar with that error... A quick websearch wasn't enlightening either.

You are in Filer's folder when issueing that command? Anyway, let's wait for owenca to see if he agrees to revert that part in the code.

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Sorry: I'm not familiar at all with Git :-)
Hower, sure: I was in Filer's folder when issueing that command.

Anyway, to summarize the trouble: despite the fact that I deleted my old AutoFilerFolders file, I cannot add any folder to watch inside Filer: at the next run, Filer will always crash.

@humdingerb
Copy link
Member

Yes. I think I found the breaking code change, as I referred to above.

If you cannot revert the code revision with "git reset", you can edit RefStorage.cpp directly.
Just replace the lines 85 and 86 with line 84, see:
514d5dd#diff-da3380bb1f27a8865f580a85f8856f9e

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

With your suggestion to fix it, now works fine (I mean with the old AutoFilerFolders file); however when I launch Filer is still in english: I am sure that I followed everything: I edit both makefile (for Filer and Autofiler) at the line:
LOCALES = it
Then I run make && make bindcatalogs and was ok since I got
for lc in it; do linkcatkeys -o objects.x86-cc2-release/Filer -s application/x-vnd.dw-Filer -tr -l $lc locales/$lc.catkeys; done
And
for lc in it; do linkcatkeys -o objects.x86-cc2-release/AutoFiler -s application/x-vnd.dw-AutoFiler -tr -l $lc locales/$lc.catkeys; done

Maybe is due to the fact that I run Haiku in English?

@humdingerb
Copy link
Member

Yes. You have to set Italian in the Locale prefs.
You normally just add "it" to the existing locales, i.e. en and de currently.
AutoFiler's makefile doesn't have to be "localized", because the it's just a daemon calling Filer, so it has no strings that need translating.

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Now I noticed that despite the fact that all my rules and folders are still recognized, AutoFiler no longer works: just Filer works: in fact if i drag in the dropzone a file that I know is recognized by a rule, all is ok. But AutoFiler is no longer triggered.

@humdingerb
Copy link
Member

Make sure you have not an "old" AutoFiler running. Best remove any other Filer or AutoFiler from your system. Filer expects AutoFiler to sit in the same folder as the Filer executable. You have to build AutoFiler too with "make -f Makefile_AutoFiler". Then both Filer and Autofiler should sit in the "objects.x86-cc2-release" folder.

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

I have the new AutoFiler running and is in the same folder of Filer; the old binaries are inside a zip file :-)

@humdingerb
Copy link
Member

Mysterious. Are you sure AutoFiler is running? Have you checked the AutoFiler tab, the button should say "Stop AutoFiler".
I tried it and AutoFiler works here...

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Yeah: I can see it in the ProcessController; in the AutoFiler tab I am able to stop and start AutoFiler.
Dis you tried using old config files?

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

I deleted again the AutoFilerFolders file, then I added again folders (well, I started by adding just one folder), and AutoFiler - the new one - still crashed...

@humdingerb
Copy link
Member

Strange. Works here...
But you are right, when I installed the old filer package and had the settings and AutoFilerFolders created with it. The new Filer showed an empty AutoFiler tab. So that setting at least, isn't backwards compatible.

If you add a "TRUE" at "DEBUGGER =" in the Makefiles of both Filer and AutoFiler, the stack trace is more meaningful. You have to re-make both Filer and AutoFiler. They apper in a new "debug" object folder.

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Done. Built both binarie with "DEBUGGER = TRUE"; however if I open the syslog file and I will look for "filer" the only entry is "KERN: debug_server: Killing team 699 (/boot/home/config/non-packaged/apps/Filer/AutoFiler)"

@humdingerb
Copy link
Member

(/boot/home/config/non-packaged/apps/Filer/AutoFiler is the newl build one, right? Just to make sure...

You should get more info by clicking on "Save report" or what the button label is on the crash alert.

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Sure. Here the debug report with debug enabled: http://sprunge.us/JICM
I say a reference to RefStorage.cpp:106, AutoFiler.cpp:34 and AutoFiler.cpp:221

@humdingerb
Copy link
Member

Thanks. Let's wait for @owenca , he's the coding boss. I just work here... :)

@Giova84
Copy link
Author

Giova84 commented Aug 2, 2017

Without your work I would have been unable to give useful (I hope) informations :-)

owenca added a commit to owenca/Filer that referenced this issue Aug 2, 2017
Reverts previous changes that allowed gcc5 linker to succeed.

Fixes HaikuArchives#95
owenca added a commit to owenca/Filer that referenced this issue Aug 2, 2017
Reverts previous changes that allowed gcc5 linker to succeed.
Fixes HaikuArchives#95

Also, make AutoFiler pass gcc5 linker.
Fixes HaikuArchives#97
@owenca
Copy link
Contributor

owenca commented Aug 3, 2017

Sorry about the chaos! Hope PR #96 will take care of it.

humdingerb pushed a commit that referenced this issue Aug 3, 2017
Reverts previous changes that allowed gcc5 linker to succeed.
Fixes #95

Also, make AutoFiler pass gcc5 linker.
Fixes #97
humdingerb pushed a commit to humdingerb/Filer that referenced this issue Aug 12, 2017
Reverts previous changes that allowed gcc5 linker to succeed.
Fixes HaikuArchives#95

Also, make AutoFiler pass gcc5 linker.
Fixes HaikuArchives#97
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants